Re: [PATCH] Clarify semantics of virDomainMigrate2

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

 



On Fri, Oct 14, 2011 at 13:40:07 -0600, Eric Blake wrote:
> On 10/14/2011 01:34 PM, Jiri Denemark wrote:
> > Explicitly disallow conflicts between domain name from dxml and dname.
> > ---
> >   src/libvirt.c             |    3 +++
> >   src/qemu/qemu_driver.c    |    4 ++--
> >   src/qemu/qemu_migration.c |   17 ++++++++++++++---
> >   src/qemu/qemu_migration.h |    1 +
> >   4 files changed, 20 insertions(+), 5 deletions(-)
> >
> > diff --git a/src/libvirt.c b/src/libvirt.c
> > index f1e6a6b..69cffc5 100644
> > --- a/src/libvirt.c
> > +++ b/src/libvirt.c
> > @@ -4854,6 +4854,9 @@ error:
> >    * in accessing the underlying storage.  The migration will fail
> >    * if @dxml would cause any guest-visible changes.  Pass NULL
> >    * if no changes are needed to the XML between source and destination.
> > + * @dxml cannot be used to rename the domain during migration (use
> > + * @dname for that purpose).  Domain name in @dxml must either match the
> > + * original domain name or @dname if it was specified.
> 
> ACK.

Thanks, I made the suggested change, marked the error message for translation
(I'm glad we have syntax-check) and pushed.

Jirka

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