]> xenbits.xensource.com Git - people/aperard/osstest.git/commitdiff
libvirt: Check /capabilities/host/migration_features/live for live migration
authorIan Jackson <ian.jackson@eu.citrix.com>
Thu, 6 Oct 2016 16:38:29 +0000 (17:38 +0100)
committerIan Jackson <Ian.Jackson@eu.citrix.com>
Fri, 14 Oct 2016 17:30:24 +0000 (18:30 +0100)
libvirt is capable of advertising this separately from
/capabilities/host/migration_features, so if save/restore is supported
but live migration is not, this will do the right thing.

We would have preferred libvirt to advertise
  /capabilities/host/migration_features/save
or something, but it doesn't right now, so we continue to use
  /capabilities/host/migration_features
to detect save/restore support.

If libvirt changes its feature presentation, then at some future point
we should change osstest too.

Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
CC: Martin Kletzander <mkletzan@redhat.com>
CC: Jim Fehlig <jfehlig@suse.com>
---
v3: Call correct function name.

Osstest/Toolstack/libvirt.pm

index 57e5b1f5f039cb0fc795d90be3afda02ae78ea4c..9d41a5257b8fd429c88d5a067a8cf13f723e3a7a 100644 (file)
@@ -98,7 +98,8 @@ sub migrate_check ($$) {
         # local migration is not supported
         $rc = 1;
     } else {
-       $rc = $self->_check_capability('/capabilities/host/migration_features');
+       $rc = $self->_check_capability
+           ('/capabilities/host/migration_features/live');
     }
 
     logm("rc=$rc");