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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 09/07/15 14:13, Ján Tomko wrote:
> 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
> 
Thanks, now pushed.
Erik

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]