Skip to content

Commit

Permalink
v2v: -o rhev, -o vdsm: Set DefaultDisplayType back to 1 (RHBZ#1260590).
Browse files Browse the repository at this point in the history
In commit c4bc811, the DefaultDisplayType
was changed from 1 -> 2.  However this is in fact wrong.  See
https://bugzilla.redhat.com/show_bug.cgi?id=1260590#c17 for details.

This reverts the code back to what it was in
commit 829e3fe.

Thanks: Omer Frenkel
  • Loading branch information
rwmjones committed Sep 9, 2015
1 parent 2b07f87 commit ab89c9d
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions v2v/OVF.ml
Expand Up @@ -282,11 +282,8 @@ let rec create_ovf source targets guestcaps inspect
e "IsStateless" [] [PCData "False"];
e "Origin" [] [PCData (string_of_int origin)];
e "VmType" [] [PCData vmtype];
(* The documentation for DefaultDisplayType is wrong. See
* https://bugzilla.redhat.com/show_bug.cgi?id=1260590#c7 for
* correct information.
*)
e "DefaultDisplayType" [] [PCData "2" (* qxl *)];
(* See https://bugzilla.redhat.com/show_bug.cgi?id=1260590#c17 *)
e "DefaultDisplayType" [] [PCData "1"];

e "Section" ["ovf:id", vm_uuid; "ovf:required", "false";
"xsi:type", "ovf:OperatingSystemSection_Type"] [
Expand Down

0 comments on commit ab89c9d

Please sign in to comment.