Merge. AFAIK, it's already merged to Master Retest? I'm out of the loop. I'm working on adding Hyper-V to Master, and not the current release.
> -----Original Message----- > From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] > Sent: 28 June 2013 3:57 PM > To: dev@cloudstack.apache.org > Subject: Re: AttachVolumeCommand no longer in use? > > 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 > <donal.laffe...@citrix.com>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:mike.tutkow...@solidfire.com] > > > Sent: 28 June 2013 3:30 PM > > > To: John Burwell; dev@cloudstack.apache.org > > > 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: mike.tutkow...@solidfire.com > > > 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: mike.tutkow...@solidfire.com > o: 303.746.7302 > Advancing the way the world uses the > cloud<http://solidfire.com/solution/overview/?video=play> > *(tm)*
