Re: [PATCH v3] qemu: Transfer inactive XML among cookie

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

 



On 16.09.2011 17:20, Michal Privoznik wrote:
> On 16.09.2011 16:53, Daniel P. Berrange wrote:
>> On Fri, Sep 16, 2011 at 08:35:15AM -0600, Eric Blake wrote:
>>> On 09/16/2011 03:51 AM, Michal Privoznik wrote:
>>>> If a domain has inactive XML we want to transfer it to destination
>>>> when migrating with VIR_MIGRATE_PERSIST_DEST. In order to harm
>>>> the migration protocol as least as possible, a optional cookie was
>>>> chosen.
>>>> ---
>>>> diff to v2:
>>>> -Fix substitution on wrong place
>>>>  src/qemu/qemu_migration.c |   81 ++++++++++++++++++++++++++++++++++++++++++---
>>>>  1 files changed, 76 insertions(+), 5 deletions(-)
>>>
>>> Looks right to me, but I'd feel more comfortable with a second ACK
>>> from danpb.  I'm also hesitant on whether we should apply this now,
>>> or wait to post-0.9.5.  While it is fixing a design flaw, it is not
>>> a regression fix, and feels more like a feature addition - if Dan
>>> could run this through his TCK suite to ensure no migration
>>> regressions, I'd feel a bit better about taking it now.
>>
>> I tend to agree. The code looks reasonable, but I think we should wait
>> till post 0.9.5, to give more time for testing. In particular the
>> interoperability beteen different versions of libvirt when doing the
>> migration
>>
>> Daniel
> 
> Okay. Fair enough. Dan, if you'll have time and run it through your TCK,
> that would be great.
> 
> I'll hold on pushing this untill 0.9.5 is released.
> 
> Michal
> 
> --
> libvir-list mailing list
> libvir-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/libvir-list

So I went ahead and pushed. Thanks.

Michal

--
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]