On Mon, Jul 13, 2015 at 12:23:52PM +0100, Ian Campbell wrote: > On Sun, 2015-07-12 at 17:20 +0100, Wei Liu wrote: > > Perhaps the libvirt part of the check_for_command stuff ought to be > moved here? Otherwise we are claiming support before the code is > actually willing to try to do so.
I move this patch before "toolstack: distinguish local and remote migration support" (the patch that claims support). It should be fine now. Wei. > > > Signed-off-by: Wei Liu <wei.l...@citrix.com> > > Cc: Ian Campbell <ian.campb...@citrix.com> > > Cc: Ian Jackson <ian.jack...@eu.citrix.com> > > Acked-by: Ian Campbell <ian.campb...@citrix.com> > > --- > > Osstest/Toolstack/libvirt.pm | 11 ++++++++--- > > 1 file changed, 8 insertions(+), 3 deletions(-) > > > > diff --git a/Osstest/Toolstack/libvirt.pm b/Osstest/Toolstack/libvirt.pm > > index ddf84df..3dc1856 100644 > > --- a/Osstest/Toolstack/libvirt.pm > > +++ b/Osstest/Toolstack/libvirt.pm > > @@ -105,17 +105,22 @@ sub saverestore_check ($) { > > > > sub migrate ($) { > > my ($self,$gho,$dst,$timeout) = @_; > > - die "Migration is not yet supported on libvirt."; > > + my $ho = $self->{Host}; > > + my $gn = $gho->{Name}; > > + target_cmd_root($ho, "virsh migrate $gn $dst", $timeout); > > } > > > > sub save ($$$$) { > > my ($self,$gho,$f,$timeout) = @_; > > - die "Save is not yet supported on libvirt."; > > + my $ho = $self->{Host}; > > + my $gn = $gho->{Name}; > > + target_cmd_root($ho, "virsh save $gn $f", $timeout); > > } > > > > sub restore ($$$$) { > > my ($self,$gho,$f,$timeout) = @_; > > - die "Restore is not yet supported on libvirt."; > > + my $ho = $self->{Host}; > > + target_cmd_root($ho, "virsh restore $f", $timeout); > > } > > > > 1; > _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel