On 09/19/2017 07:23 PM, Cole Robinson wrote:
> On 09/19/2017 10:18 AM, Pavel Hrdina wrote:
>> The current order of check operations is wrong.  First we set the new
>> path for the disk in question and after that we check whether some
>> guest already uses a disk with the same path.
>>
>> The issue is that virt-manager returns a cached list of guests in
>> path_in_use_by() and the cached guest has the disk path already
>> updated.
>>
>> Ideally we don't update the path at all, but we do it to run some
>> checks before the path is actually changed.  In order to fix the
>> referenced bug, change the order of check operations.
>>
>> Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1453094
>>
>> Signed-off-by: Pavel Hrdina <[email protected]>
> 
> ACK
> 

Since I'm cutting a release shortly, I've pushed this

Thanks,
Cole

_______________________________________________
virt-tools-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/virt-tools-list

Reply via email to