On Friday, 6 April 2018 17:45:22 CEST Tomáš Golembiovský wrote:
> On Thu,  5 Apr 2018 14:53:51 +0200
> Pino Toscano <[email protected]> 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

Attachment: signature.asc
Description: This is a digitally signed message part.

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

Reply via email to