[libvirt] [PATCH] Migration with --live --offline option performs offline migration without warning/error.
Nitesh Konkar
niteshkonkar.libvirt at gmail.com
Mon Feb 22 12:33:42 UTC 2016
Hello Peter,
Thanks for the reply. I have made changes according to your suggestion and
re-submitted the patch as [patch v2] on the mailing list.
Link:-
https://www.redhat.com/archives/libvir-list/2016-February/msg01071.html
Warm Regards,
Nitesh Konkar.
On Fri, Feb 19, 2016 at 8:57 PM, Peter Krempa <pkrempa at redhat.com> wrote:
> On Fri, Feb 19, 2016 at 07:24:17 -0500, Nitesh Konkar wrote:
> >
> > This patch gives an error when migration is attempted with both
> > --live and --offline options.
> >
> > Signed-off-by: Nitesh Konkar <nitkon12 at linux.vnet.ibm.com>
> > ---
> > tools/virsh-domain.c | 6 ++++++
> > 1 file changed, 6 insertions(+)
> >
> > diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c
> > index 62acecb..527db38 100644
> > --- a/tools/virsh-domain.c
> > +++ b/tools/virsh-domain.c
> > @@ -9795,6 +9795,12 @@ doMigrate(void *opaque)
> > } else {
> > /* For traditional live migration, connect to the destination
> host directly. */
> > virDomainPtr ddom = NULL;
> > +
> > + if ((flags & VIR_MIGRATE_OFFLINE) && (flags &
> VIR_MIGRATE_LIVE)){
> > + vshError(ctl, "%s", _("migrate: Using both --live and
> --offline option together is not permitted."));
> > + goto out;
> > + }
>
>
> This should be validated in the callers using one of the
> VSH_EXCLUSIVE_OPTIONS_* macros.
>
> Peter
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20160222/752334b5/attachment-0001.htm>
More information about the libvir-list
mailing list