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

[Yaniv Kaul]

On Thu, Nov 30, 2017 at 11:38 AM, Arik Hadas <ahadas at redhat.com> wrote:

>
>
> 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..
>

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.
Y.


>
>>
>>
>>>
>>> 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/017843d8/attachment.html>