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: deleteVolume is sync
Date Thu, 06 Jun 2013 16:14:58 GMT
If it's a long-running op to delete a volume (which is can be), I would say
it should be async.


On Thu, Jun 6, 2013 at 9:25 AM, Marcus Sorensen <shadowsor@gmail.com> wrote:

> So does it just need to be async, or is deleteVolume doing too much in
> both moving the volume to destroy state and expunging? If I transition
> a volume to 'Destroy' state, the storage cleanup thread comes along
> and deletes it for me later, similar to how the VMs are expunged. This
> seems preferable, because one could potentially undelete a volume
> within the window.
>
> On Wed, Jun 5, 2013 at 7:47 PM, Mike Tutkowski
> <mike.tutkowski@solidfire.com> wrote:
> > Hey Marcus,
> >
> > To me, it seems like it should be async, as well.
> >
> > As far as I know (at least in pre 4.2), unless you are deleting a volume
> > that has never been attached to a VM, the CS MS would have to have the
> > hypervisor perform some operation upon the deletion of a CloudStack
> > volume...and that could take a bit of time.
> >
> >
> >
> >
> > On Wed, Jun 5, 2013 at 7:24 PM, Marcus Sorensen <shadowsor@gmail.com>
> wrote:
> >
> >> Oh, I should add that I traced it through the system, and it actually
> >> sends a DeleteVolumeCommand to the agent. That has to finish before
> >> the sync call completes.
> >>
> >> This is on 4.1, if it changes significantly with the storage refactor,
> >> that's fine, but I'd like to know if there was a reason for it in case
> >> we want to make it async for us.
> >>
> >> On Wed, Jun 5, 2013 at 7:21 PM, Marcus Sorensen <shadowsor@gmail.com>
> >> wrote:
> >> > Just wondering why deleteVolume is a sync call. It doesn't seem to
> >> > adhere to the 'mark it removed, let a worker expunge it later after X
> >> > seconds' paradigm.  I only noticed this when a storage system was
> >> > taking a bit to do the work and thus blocking the API call.
> >>
> >
> >
> >
> > --
> > *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>
> > *™*
>



-- 
*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