[libvirt] [dbus PATCH v2] main: add support for all usable libvirt drivers

[Daniel P. Berrange]

On Tue, Jan 23, 2018 at 02:11:50PM +0100, Pavel Hrdina wrote:
> Some of the drivers are remote only, such as ESX, GSX, VPX, PHyp and
> HyperV.  Currently we support only local drivers.
> 
> Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
> ---
>  src/main.c          | 21 +++++++++++++++++----
>  test/libvirttest.py |  2 +-
>  2 files changed, 18 insertions(+), 5 deletions(-)
> 
> diff --git a/src/main.c b/src/main.c
> index ab03bc1..32c1cc4 100644
> --- a/src/main.c
> +++ b/src/main.c
> @@ -85,13 +85,26 @@ struct virtDBusDriver {
>  };
>  
>  static const struct virtDBusDriver sessionDrivers[] = {
> -    { "qemu:///session",    "/org/libvirt/qemu" },
> -    { "test:///default",    "/org/libvirt/test" },
> +    { "qemu:///session",            "/org/libvirt/QEMU" },
> +    { "test:///default",            "/org/libvirt/Test" },
> +    { "uml:///session",             "/org/libvirt/UML" },
> +    { "vbox:///session",            "/org/libvirt/VBox" },
> +    { "vmwarefusion:///session",    "/org/libvirt/VMwareFusion" },
> +    { "vmwareplayer:///session",    "/org/libvirt/VMwarePlayer" },
> +    { "vmwarews:///session",        "/org/libvirt/VMwareWS" },
>  };
>  
>  static const struct virtDBusDriver systemDrivers[] = {
> -    { "qemu:///system",     "/org/libvirt/qemu" },
> -    { "test:///default",    "/org/libvirt/test" },
> +    { "XenApi://localhost/",    "/org/libvirt/XenAPI" },

Now that I think some more, I have a feeling XenAPI servers are
a blackbox, so would be impossible to use localhost for that
too. We can always add it back later if I turn out to be wrong.

> +    { "bhyve:///system",        "/org/libvirt/BHyve" },
> +    { "lxc:///",                "/org/libvirt/LXC" },
> +    { "openvz:///system",       "/org/libvirt/OpenVZ" },
> +    { "qemu:///system",         "/org/libvirt/QEMU" },
> +    { "test:///default",        "/org/libvirt/Test" },
> +    { "uml:///system",          "/org/libvirt/UML" },
> +    { "vbox:///system",         "/org/libvirt/VBox" },
> +    { "vz:///system",           "/org/libvirt/VZ" },
> +    { "xen:///",                "/org/libvirt/Xen" },
>  };

If you remove XenAPI

Reviewed-by: Daniel P. Berrange <berrange at redhat.com>



Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|