On Wed, May 25, 2011 at 12:12:46PM +0100, Daniel P. Berrange wrote: > On Wed, May 25, 2011 at 11:55:08AM +0100, Matthew Booth wrote: > > volDelete used to return VIR_ERR_INTERNAL_ERROR when attempting to delete a > > volume which was still being allocated. It should return > > VIR_ERR_OPERATION_INVALID. > > > > * src/storage/storage_driver.c: Fix return of volDelete. > > --- > > src/storage/storage_driver.c | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/src/storage/storage_driver.c b/src/storage/storage_driver.c > > index 2da2feb..d9c2137 100644 > > --- a/src/storage/storage_driver.c > > +++ b/src/storage/storage_driver.c > > @@ -1914,7 +1914,7 @@ storageVolumeDelete(virStorageVolPtr obj, > > } > > > > if (vol->building) { > > - virStorageReportError(VIR_ERR_INTERNAL_ERROR, > > + virStorageReportError(VIR_ERR_OPERATION_INVALID, > > _("volume '%s' is still being allocated."), > > vol->name); > > goto cleanup; > > ACK Okay, I finally commited this, Daniel -- Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ daniel@xxxxxxxxxxxx | Rpmfind RPM search engine http://rpmfind.net/ http://veillard.com/ | virtualization library http://libvirt.org/ -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list