[libvirt] [PATCH] Don't replace persistent domain config with migrated config

[Eric Blake]

On 03/24/2010 08:52 AM, Jiri Denemark wrote:
> When a domain is defined on host1, migrated to host2 and then migrated
> back to host1, its current configuration would overwrite the libvirtd's
> in-memory copy of persistent configuration of that domain. This is not
> desired as we want to preserve the persistent configuration untouched.
> 
> This patch introduces new 'live' parameter to virDomainAssignDef.
> Passing 'true' for 'live' means the configuration passed to
> virDomainAssignDef describes a configuration of live instance of the
> domain. This applies for saved domains which are being restored or for
> incoming domains during migration.

ACK.

> 
>  virDomainObjPtr virDomainAssignDef(virCapsPtr caps,
...

>          } else {
> -            if (domain->newDef)
> -                virDomainDefFree(domain->newDef);
> +            virDomainDefFree(domain->newDef);

And nice removal of useless-if-before-free.

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 323 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100325/cb91dd82/attachment-0001.sig>