On Friday, 6 April 2018 17:45:22 CEST Tomáš Golembiovský wrote:
 On Thu,  5 Apr 2018 14:53:51 +0200
 Pino Toscano <ptoscano(a)redhat.com> wrote:
 
 > @@ -383,8 +581,9 @@ let rec create_ovf source targets guestcaps inspect
 >          ] in
 >          (match ovf_flavour with
 >          | OVirt ->
 > -          e "OperatingSystemSection" ["ovf:id", vm_uuid;
 > -                                      "ovf:required", "false"]
 > +          let ovirt_osid = get_ovirt_osid inspect in
 > +          e "OperatingSystemSection" ["ovf:required",
"false";
 > +                                      "ovirt:id", string_of_int
ovirt_osid]
 
 Sadly, OVF schema says ovf:id is reqiured. But since the generated OVF
 is not used by anything besides oVirt I think we can live without it. 
Not a problem them: I did put it back, and push the patch.
Thanks,
-- 
Pino Toscano