[PATCH virt-install 2/2] Don't generate port and tlsport parameters in xml if spice gl is enabled

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



spice gl + tcp port is disallowed by qemu commit 569a93c, So in this
case we don't generate port and tlsport parameters in domain xml.

Signed-off-by: Lin Ma <lma@xxxxxxxx>
---
 tests/cli-test-xml/compare/virt-install-spice-gl.xml | 2 +-
 virtinst/guest.py                                    | 6 ++++++
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/tests/cli-test-xml/compare/virt-install-spice-gl.xml b/tests/cli-test-xml/compare/virt-install-spice-gl.xml
index 44a6700..0407cd2 100644
--- a/tests/cli-test-xml/compare/virt-install-spice-gl.xml
+++ b/tests/cli-test-xml/compare/virt-install-spice-gl.xml
@@ -46,7 +46,7 @@
       <model type="virtio"/>
     </interface>
     <input type="tablet" bus="usb"/>
-    <graphics type="spice" port="-1" tlsPort="-1" autoport="yes">
+    <graphics type="spice">
       <gl enable="yes"/>
       <image compression="off"/>
     </graphics>
diff --git a/virtinst/guest.py b/virtinst/guest.py
index ab6dcba..e92c24a 100644
--- a/virtinst/guest.py
+++ b/virtinst/guest.py
@@ -1132,6 +1132,12 @@ class Guest(XMLBuilder):
             if dev.type == "default":
                 _set_type(dev)
 
+            # spice gl + tcp port is disallowed by qemu commit 569a93c.
+            # So we dont generate port/tlsport parameters in domain xml.
+            if dev.type == "spice" and self.has_gl():
+                dev.port = None
+                dev.tlsPort = None
+
             if (dev.type == "spice" and
                 not self.conn.is_remote() and
                 self.conn.check_support(
-- 
2.8.1

_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list



[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux