[virt-tools-list] [PATCH] virt-manager: drop 'xenmigr' scheme from Xen migration URI

Cole Robinson crobinso at redhat.com
Thu Sep 8 18:34:50 UTC 2016


On 09/07/2016 12:55 AM, Jim Fehlig wrote:
> For Xen, virt-manager uses a 'xenmigr' URI scheme, which is not
> supported by the libvirt libxl driver. Attempting migration
> fails with
> 
> libvirtError: invalid argument: unable to parse URI: xenmigr://myhost
> 
> The old xend-based libvirt driver supports this scheme, but also
> supports an empty scheme. It's not clear what the 'xenmigr' scheme
> is used for. 'xenmigr' is not referenced by any files in the Xen
> code-base, including old branches with xend.
> 
> Drop setting scheme to 'xenmigr' when creating the Xen migration URI.
> 
> Signed-off-by: Jim Fehlig <jfehlig at suse.com>
> ---
>  virtManager/migrate.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/virtManager/migrate.py b/virtManager/migrate.py
> index ca430dd..9601969 100644
> --- a/virtManager/migrate.py
> +++ b/virtManager/migrate.py
> @@ -344,7 +344,7 @@ class vmmMigrateDialog(vmmGObjectUI):
>              return
>  
>          if self.conn.is_xen():
> -            uri = "xenmigr://%s" % address
> +            uri = "%s" % address
>          else:
>              uri = "tcp:%s" % address
>          if port:
> 

Thanks, pushed now

- Cole




More information about the virt-tools-list mailing list