[ovirt-devel] oVirt 4.2.0 blockers review - Day 3

[Michal Skrivanek]

> On 30 Nov 2017, at 05:13, Yaniv Kaul <ykaul at redhat.com> wrote:
> 
> 
> 
> On Thu, Nov 30, 2017 at 11:38 AM, Arik Hadas <ahadas at redhat.com <mailto:ahadas at redhat.com>> wrote:
> 
> 
> On Thu, Nov 30, 2017 at 11:10 AM, Martin Sivak <msivak at redhat.com <mailto:msivak at redhat.com>> wrote:
> 
> 
> Bug ID	Product	Assignee	Status	Summary
> 1518887 <https://bugzilla.redhat.com/show_bug.cgi?id=1518887>	ovirt-hosted-engine-ha	bugs at ovirt.org <mailto:bugs at ovirt.org>	NEW	ovirt-ha-agent fails parsing the OVF_STORE due to a change in OVF namespace URI
> 
> I'm in favor of reverting the virt change personally.
> 
> Unless something else depends on it, the commit message said vdsm needs this.
> 
> vdsm needs it because we would like vdsm to be able to parse OVA files that are generated by us and OVA files that  are generated by others (and are VMware-compatbile) with the existing code in vdsm. The existing code is tailored to VMware-compatible OVA files that are generated by others, in which the uri doesn't include that slash at the end.
> 
> It would be best to adjust ovirt-ha-agent to parse the right uri.
> However, it that's too complicated, an alternative solution is to keep writing the previous uri to ovirt's OVFs, i.e., those in OVF_STORE and in snapshot's configuration. That would be a pitty since we want to minimize the differences between the OVFs we generate, but it would be better than reverting the change..
> 
> I think a reasonable alternative is to revert the change, adjust HE to use both formats, and then introduce the new format. The first step (revert) should be done for 4.2.0 (today?) and the rest for 4.2.1.

It shouldn’t take too long to polish [1] and rebuild ovirt-hosted-engine-ha
Thanks Martin for posting it!

Thanks,
michal

[1] https://gerrit.ovirt.org/#/c/84933/

> Y.
> 
>  
>  
> 
> 1516113 <https://bugzilla.redhat.com/show_bug.cgi?id=1516113>	cockpit-ovirt	phbailey at redhat.com <mailto:phbailey at redhat.com>	POST	Deploy the HostedEngine failed with the default CPU type
> 
> Would be happy if the remaining patch could get reviewed quickly.
> 
> 
>  
> 
> 1518693 <https://bugzilla.redhat.com/show_bug.cgi?id=1518693>	ovirt-engine	akrejcir at redhat.com <mailto:akrejcir at redhat.com>	POST	Quota is needed to copy template disk
> 
> This is only via REST and the default quota can be used as a workaround - why is this a blocker?
> 
> Automation added it because Raz marked it as Regression. But the change was intentional.
>  
>  
> 1517810 <https://bugzilla.redhat.com/show_bug.cgi?id=1517810>	ovirt-engine	stirabos at redhat.com <mailto:stirabos at redhat.com>	Adding additional ha-host fails.
> 
> This one has a verified engine fix now, we just need to merge it and update Node 0 setup (also verified).
> 
> 
> Martin
> 
> 
> _______________________________________________
> Devel mailing list
> Devel at ovirt.org <mailto:Devel at ovirt.org>
> http://lists.ovirt.org/mailman/listinfo/devel <http://lists.ovirt.org/mailman/listinfo/devel>
> 
> 
> _______________________________________________
> Devel mailing list
> Devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/devel

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ovirt.org/pipermail/devel/attachments/20171130/9e992645/attachment-0001.html>