cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mike Tutkowski <mike.tutkow...@solidfire.com>
Subject Re: AttachVolumeCommand no longer in use?
Date Fri, 28 Jun 2013 16:56:39 GMT
Oh, yeah, to clarify, I just meant I need to continue to merge these large
changes into my code and then re-test my code before I commit it.


On Fri, Jun 28, 2013 at 9:23 AM, Donal Lafferty
<donal.lafferty@citrix.com>wrote:

> 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.tutkowski@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.lafferty@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.tutkowski@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.tutkowski@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.tutkowski@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.tutkowski@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloud<http://solidfire.com/solution/overview/?video=play>
*™*

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message