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

[Arik Hadas]

On Thu, Nov 30, 2017 at 11:10 AM, Martin Sivak <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 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 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 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 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
> http://lists.ovirt.org/mailman/listinfo/devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ovirt.org/pipermail/devel/attachments/20171130/91f46582/attachment-0001.html>