Re: [libvirt] [PATCH 1/3] add virCond into _virStoragePoolObj

[Dave Allan <dallan redhat com>]

On Mon, Jul 11, 2011 at 06:11:47PM +0800, Guannan Ren wrote:
> On 07/11/2011 04:49 PM, Jiri Denemark wrote:
> >On Sun, Jul 10, 2011 at 22:29:03 +0800, Guannan Ren wrote:
> >>The three patches add support to delete volumes while it is building intead of raising
> >>an warning infomation,then abort.
> >>For the volume of raw file, delete it directly without any effect on the thread of creating
> >>it.
> >>For the volume of block, wait for its finish , then delete it.
> >Hmm, shouldn't we cancel volume creation instead of waiting for it to finish?
> >
> >Jirka
>        Yes, that is expected, but for volumes in pools of disk and
> logical type, it is
>     comparatively safe way to delete after its finish as the
> creation of these volume
>     is changing the files system or something low-level.

No, IMO, we really need to figure out if the operation is something
that can be cancelled, and if so, cancel it.  

Dave