Re: [PATCH OSSTEST] Toolstack/libvirt: use URI in migration command

[Jim Fehlig <jfehlig@suse.com>]

Wei Liu wrote:
> Virsh migrate expects an URI, not a host. We don't actually care what
> kind of transport it uses, the main objective is to test migration, so
> use xen+ssh for the time being.
>
> Signed-off-by: Wei Liu <wei.liu2@citrix.com>
> Cc: Ian Campbell <ian.campbell@citrix.com>
> Cc: Ian Jackson <ian.jackson@eu.citrix.com>
> ---
> Cc: Jim Fehlig <jfehlig@suse.com>
>
> I once had email exchange with Jim and he said xen+ssh was a commonly
> use configuration.
>
> Jim, I hope I got the URI right. The constructed command will be like
>
>    virsh migrate guest xen+ssh://dst-host
>   

Yes, that is correct. Note you will need a '--live' option if live
migration is what you are after

  virsh migrate --live guest xen+ssh://dst-host

Regards,
Jim
> .
>
> OSSTest has working ssh setup between src and dst host so I expect this
> setup to just work.
> ---
>  Osstest/Toolstack/libvirt.pm | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Osstest/Toolstack/libvirt.pm b/Osstest/Toolstack/libvirt.pm
> index bfb5ccb..477ebee 100644
> --- a/Osstest/Toolstack/libvirt.pm
> +++ b/Osstest/Toolstack/libvirt.pm
> @@ -108,7 +108,7 @@ sub migrate ($$$$) {
>      my $sho = $self->{Host};
>      my $dst = $dho->{Name};
>      my $gn = $gho->{Name};
> -    target_cmd_root($sho, "virsh migrate $gn $dst", $timeout);
> +    target_cmd_root($sho, "virsh migrate $gn xen+ssh://$dst", $timeout);
>  }
>  
>  sub save ($$$$) {
>