Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cloudstack.apache.org
Received-SPF: pass (athena.apache.org: domain of mike.tutkowski@solidfire.com
 designates 209.85.219.47 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <9ADDE3F979256644BED8F0D244BE51F0034997@LONPEX01CL02.citrite.net>
References: 
 <CACOnxCjVDe3HY6Hy6H9DZsGdkjxJkmmxnFAC5pgCeZ12n99rdg@mail.gmail.com>
	<9ADDE3F979256644BED8F0D244BE51F0034962@LONPEX01CL02.citrite.net>
	<CACOnxCjq0DpJ8LR+iq28r1fdBB9E8b95_GRhi=M9EsKbrcu6xg@mail.gmail.com>
	<9ADDE3F979256644BED8F0D244BE51F0034997@LONPEX01CL02.citrite.net>
Date: Fri, 28 Jun 2013 10:56:39 -0600
Message-ID: 
 <CACOnxCjdn06nWu6xtOQ7+QO4jyiEHix2wYueSkbD0msD_xW=qQ@mail.gmail.com>
Subject: Re: AttachVolumeCommand no longer in use?
From: Mike Tutkowski <mike.tutkowski@solidfire.com>
To: "dev@cloudstack.apache.org" <dev@cloudstack.apache.org>
Content-Type: multipart/alternative; boundary=089e0111dd44392b2704e039c398

--089e0111dd44392b2704e039c398
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: quoted-printable

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=3Dplay>
> > > > *(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=3Dplay>
> > *(tm)*
>


--=20
*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=3Dplay>
*=99*

--089e0111dd44392b2704e039c398--