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

On Thu, Nov 30, 2017 at 11:10 AM, Martin Sivak <msivak(a)redhat.com&gt; wrote: > >>> >>> Bug ID Product Assignee Status Summary >>> 1518887 <https://bugzilla.redhat.com/show_bug.cgi?id=1518887> >>> ovirt-hosted-engine-ha bugs(a)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..

> > >> >> 1516113 <https://bugzilla.redhat.com/show_bug.cgi?id=1516113> >>> cockpit-ovirt phbailey(a)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(a)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(a)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(a)ovirt.org > http://lists.ovirt.org/mailman/listinfo/devel >