Hi Cole
On Sun, May 8, 2016 at 3:36 AM Cole Robinson <crobinso@xxxxxxxxxx> wrote:
On 04/28/2016 08:22 AM, Marc-André Lureau wrote:
> Signed-off-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
> ---
> virtinst/guest.py | 5 +++++
> virtinst/support.py | 1 +
> 2 files changed, 6 insertions(+)
>
> diff --git a/virtinst/guest.py b/virtinst/guest.py
> index ab6dcba..5824e87 100644
> --- a/virtinst/guest.py
> +++ b/virtinst/guest.py
> @@ -1141,6 +1141,11 @@ class Guest(XMLBuilder):
> if dev.image_compression is None:
> dev.image_compression = "off"
>
> + if (dev.type == "spice" and
> + not self.conn.check_support(
> + self.conn.SUPPORT_CONN_SPICE_GL)):
> + raise ValueError(_("Host does not support spice GL"))
> +
> def _add_spice_channels(self):
> if self.skip_default_channel:
> return
> diff --git a/virtinst/support.py b/virtinst/support.py
> index 1071750..1925986 100644
> --- a/virtinst/support.py
> +++ b/virtinst/support.py
> @@ -311,6 +311,7 @@ SUPPORT_CONN_VCPU_PLACEMENT = _make(
> SUPPORT_CONN_MEM_STATS_PERIOD = _make(
> function="virDomain.setMemoryStatsPeriod",
> version="1.1.1", hv_version={"qemu": 0})
> +SUPPORT_CONN_SPICE_GL = _make(version="1.3.3", hv_version={"qemu": "2.5.92"})
>
If we are going to wait for the libvirt listen type=none, maybe we can avoid
this and use libvirt domcapabilities instead. It's on my todo list to use
domcapabilities to advertise spice gl bits. We will see how this libvirt
release cycle goes
Any news regarding domcapabilities? Should we go with that patch instead for now?
Thanks,
Cole
_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list
--
Marc-André Lureau
_______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list