So, basically, it could take a while to merge and re-test this. This week has been kind of a mess with all of the merges.
Do you know if we have the weekend to tie up loose ends? On Fri, Jun 28, 2013 at 8:55 AM, Donal Lafferty <[email protected]>wrote: > Yeah, I noticed that as well. > > The implementation of AttachCommand is in XenServerStorageProcessor. Look > for handleStorageCommands in CitrixResourceBase. > > Also, could you send me a JSON encoded version of the command? > > > > > -----Original Message----- > > From: Mike Tutkowski [mailto:[email protected]] > > Sent: 28 June 2013 3:30 PM > > To: John Burwell; [email protected] > > Subject: AttachVolumeCommand no longer in use? > > > > Hi, > > > > I've been trying to reconcile a ton of changes that were merged in this > week. > > > > We used to pass an AttachVolumeCommand to CitrixResourceBase, but now > > it's called AttachCommand and CitrixResourceBase doesn't know anything > > about it (this merge issue is in VolumeManagerImpl). > > > > I'm a bit confused at the moment on how to handle this merge issue. > > > > Does anyone know anything about this change? > > > > Thanks! > > > > -- > > *Mike Tutkowski* > > *Senior CloudStack Developer, SolidFire Inc.* > > e: [email protected] > > o: 303.746.7302 > > Advancing the way the world uses the > > cloud<http://solidfire.com/solution/overview/?video=play> > > *(tm)* > -- *Mike Tutkowski* *Senior CloudStack Developer, SolidFire Inc.* e: [email protected] o: 303.746.7302 Advancing the way the world uses the cloud<http://solidfire.com/solution/overview/?video=play> *™*
