On Mon, Dec 25, 2017 at 2:20 PM, Barak Korren <bkorren(a)redhat.com> wrote:
>
>
> On 25 December 2017 at 10:50, Eyal Edri <eedri(a)redhat.com> wrote:
>
>> I'm testing a revert of the patch [1], but it looks like it is very
>> related and suspicion is around ansible not configured firewalld and thus
>> causing the failure.
>> Gal has been debugging the same issue downstream yesterday.
>>
>>
>> [1]
https://gerrit.ovirt.org/#/c/85723/
>>
>>
>>
> Revet patch failed to pass CQ:
>
http://jenkins.ovirt.org/job/ovirt-master_change-queue-tester/4518/
>
> I'm rerunning just in case it passes a 2nd time:
>
http://jenkins.ovirt.org/job/ovirt-master_change-queue-tester/4520/
>
> Maybe we need to revert the firewalld patch too:
>
https://gerrit.ovirt.org/c/85326 (engine: Let backend choose default
> firewall type)
>
OK, Thanks for Gal for debugging this further, it seems that a different
patch was causing the failure:
https://gerrit.ovirt.org/#/c/85611/ - core: ansible: Don't print errors
when files not found
I've tested a revert patch [1]( rebased on master ) and it passed here [2].
Details on why it caused the failure will be added to the revert patch
commit msg.
| TRIED. TESTED. TRUSTED. |