As of ovirt-engine commit daca2ca6cd91, ovirt-engine ignores

  <rasd:Device>qxl</rasd:Device>

entirely; ovirt-engine picks the QXL device based on other factors. In
preparation for ovirt-engine honoring <rasd:Device>, and in particular for
selecting VGA over QXL as a policy change, replace the element's contents
"qxl" with "vga". (The latter is a part of ovirt-engine's VmDeviceType
enum type just the same.)

Reference:
[Libguestfs] specifying a standard VGA video controller in OVF for oVirt
https://listman.redhat.com/archives/libguestfs/2021-November/msg00149.html

Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1961107
Signed-off-by: Laszlo Ersek <[email protected]>
---

Notes:
    v1:
    
    - new in v1

 lib/create_ovf.ml                          | 11 +++++++----
 tests/test-v2v-o-rhv.ovf.expected          |  2 +-
 tests/test-v2v-o-vdsm-options.ovf.expected |  2 +-
 3 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/lib/create_ovf.ml b/lib/create_ovf.ml
index 8c8ef43a42cd..6ccea1e9e6d7 100644
--- a/lib/create_ovf.ml
+++ b/lib/create_ovf.ml
@@ -675,9 +675,12 @@ let rec create_ovf source inspect
           e "rasd:UsbPolicy" [] [PCData "Disabled"];
         ];
 
-        (* We always add a qxl device when outputting to RHV.
-         * See RHBZ#1213701 and RHBZ#1211231 for the reasoning
-         * behind that.
+        (* We always add a standard VGA-compatible video controller when
+         * outputting to RHV. See RHBZ#1213701 and RHBZ#1211231 for the
+         * reasoning behind that. The device model used to be QXL previously,
+         * but only the unaccelerated standard VGA framebuffer is needed; so
+         * the (simpler) standard VGA device itself suffices. Refer to
+         * RHBZ#1961107.
          *)
         let monitor_resourcetype =
           match ovf_flavour with
@@ -690,7 +693,7 @@ let rec create_ovf source inspect
             [PCData (string_of_int monitor_resourcetype)];
           e "Type" [] [PCData "video"];
           e "rasd:VirtualQuantity" [] [PCData "1"];
-          e "rasd:Device" [] [PCData "qxl"];
+          e "rasd:Device" [] [PCData "vga"];
         ]
       ];
 
diff --git a/tests/test-v2v-o-rhv.ovf.expected 
b/tests/test-v2v-o-rhv.ovf.expected
index f8aa07c45d4c..a8f5ebdfc7e9 100644
--- a/tests/test-v2v-o-rhv.ovf.expected
+++ b/tests/test-v2v-o-rhv.ovf.expected
@@ -60,7 +60,7 @@
         <rasd:ResourceType>20</rasd:ResourceType>
         <Type>video</Type>
         <rasd:VirtualQuantity>1</rasd:VirtualQuantity>
-        <rasd:Device>qxl</rasd:Device>
+        <rasd:Device>vga</rasd:Device>
       </Item>
       <Item>
         <rasd:Caption>RNG Device</rasd:Caption>
diff --git a/tests/test-v2v-o-vdsm-options.ovf.expected 
b/tests/test-v2v-o-vdsm-options.ovf.expected
index f861071c08c3..a2a26a839462 100644
--- a/tests/test-v2v-o-vdsm-options.ovf.expected
+++ b/tests/test-v2v-o-vdsm-options.ovf.expected
@@ -60,7 +60,7 @@
         <rasd:ResourceType>32768</rasd:ResourceType>
         <Type>video</Type>
         <rasd:VirtualQuantity>1</rasd:VirtualQuantity>
-        <rasd:Device>qxl</rasd:Device>
+        <rasd:Device>vga</rasd:Device>
       </Item>
       <Item>
         <rasd:Caption>RNG Device</rasd:Caption>
-- 
2.19.1.3.g30247aa5d201


_______________________________________________
Libguestfs mailing list
[email protected]
https://listman.redhat.com/mailman/listinfo/libguestfs

Reply via email to