[libvirt] [PATCH] storage: Revert volume obj list updating after volume creation (4749d82a)

Ján Tomko jtomko at redhat.com
Thu Jul 9 12:13:57 UTC 2015


On Thu, Jul 09, 2015 at 01:48:16PM +0200, Erik Skultety wrote:
> This patch reverts commit 4749d82a which tried to tweak the logic in
> volume creation. We did realloc and update our object list before we executed
> volume building within a specific storage backend. If that failed, we
> had to update (again) our object list to the original state as it was before the
> build and delete the volume from the pool (even though it didn't exist - this
> truly depends on the backend).
> I misunderstood the base idea to be able to poll the status of the volume
> creation using vol-info. After commit 4749d82a this wasn't possible
> anymore, although no BZ has been reported yet.
> 
> Commit 4749d82a also claimed to fix
> https://bugzilla.redhat.com/show_bug.cgi?id=1223177, but commit c8be606b of the
> same series as 4749d82ad (which was more of a refactor than a fix)
> fixes the same issue so the revert should be pretty straightforward.
> Further more, BZ https://bugzilla.redhat.com/show_bug.cgi?id=1241454 can be
> fixed with this revert.
> ---
>  src/storage/storage_driver.c | 28 ++++++++++++++++++----------
>  1 file changed, 18 insertions(+), 10 deletions(-)

ACK

Jan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20150709/d65e7183/attachment-0001.sig>


More information about the libvir-list mailing list