[libvirt] [PATCH] Fix migration for 3.3 xend

# HG changeset patch # User John Levon <john.levon@sun.com> # Date 1234535031 28800 # Node ID 3b0d2047df2c53ff2994d2bff4b1123110626ee2 # Parent 0c02b55855c7a3bfa1135887ffb1763ef29179e5 Fix migration for 3.3 xend xend grew a couple of new options for the migration method. As they're ignored on older xend versions, always pass them. Signed-off-by: John Levon <john.levon@sun.com> diff --git a/src/xend_internal.c b/src/xend_internal.c --- a/src/xend_internal.c +++ b/src/xend_internal.c @@ -4292,6 +4292,8 @@ xenDaemonDomainMigratePerform (virDomain "destination", hostname, "live", live, "port", port, + "node", "-1", + "ssl", "0", "resource", "0", /* required, xend ignores it */ NULL); VIR_FREE (hostname);

On Fri, Feb 13, 2009 at 06:30:59AM -0800, John Levon wrote:
# HG changeset patch # User John Levon <john.levon@sun.com> # Date 1234535031 28800 # Node ID 3b0d2047df2c53ff2994d2bff4b1123110626ee2 # Parent 0c02b55855c7a3bfa1135887ffb1763ef29179e5 Fix migration for 3.3 xend
xend grew a couple of new options for the migration method. As they're ignored on older xend versions, always pass them.
ACK, surprised they weren't made optional when added :-(
diff --git a/src/xend_internal.c b/src/xend_internal.c --- a/src/xend_internal.c +++ b/src/xend_internal.c @@ -4292,6 +4292,8 @@ xenDaemonDomainMigratePerform (virDomain "destination", hostname, "live", live, "port", port, + "node", "-1", + "ssl", "0", "resource", "0", /* required, xend ignores it */ NULL); VIR_FREE (hostname);
Daniel -- |: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :| |: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|

On Fri, Feb 13, 2009 at 04:15:56PM +0000, Daniel P. Berrange wrote:
Fix migration for 3.3 xend
xend grew a couple of new options for the migration method. As they're ignored on older xend versions, always pass them.
ACK, surprised they weren't made optional when added :-(
I don't think it's even possible. It's like XML-RPC but even more painful. regards john
participants (3)
-
Daniel P. Berrange
-
John Levon
-
John Levon