<div dir="ltr">This patch fixes the problem:<br><a href="https://gerrit.ovirt.org/#/c/81094/">https://gerrit.ovirt.org/#/c/81094/</a><br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Aug 27, 2017 at 4:55 PM, Dan Kenigsberg <span dir="ltr"><<a href="mailto:danken@redhat.com" target="_blank">danken@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Sun, Aug 27, 2017 at 12:24 PM, Eyal Edri <<a href="mailto:eedri@redhat.com">eedri@redhat.com</a>> wrote:<br>
> Adding patch author and maintainers.<br>
><br>
> On Fri, Aug 25, 2017 at 11:40 AM, Anton Marchukov <<a href="mailto:amarchuk@redhat.com">amarchuk@redhat.com</a>><br>
> wrote:<br>
>><br>
>> Hello All.<br>
>><br>
>> Just noted that change queue job [1] for ovirt-provider-ovn started to<br>
>> fail consistently while the actual reason for that seems to be RPM build<br>
>> failure at [2]:<br>
>><br>
>> 14:47:42 Checking for unpackaged file(s): /usr/lib/rpm/check-files<br>
>> /home/jenkins/workspace/ovirt-<wbr>provider-ovn_master_build-<wbr>artifacts-el7-x86_64/ovirt-<wbr>provider-ovn/rpmbuild/<wbr>BUILDROOT/ovirt-provider-ovn-<wbr>1.1-2.20170824144740.<wbr>gitcf66669.el7.centos.x86_64<br>
>> 14:47:42 error: Installed (but unpackaged) file(s) found:<br>
>> 14:47:42 /usr/share/doc/ovirt-provider-<wbr>ovn/README.adoc<br>
>> 14:47:42<br>
>> 14:47:42<br>
>> 14:47:42 RPM build errors:<br>
>> 14:47:42 Installed (but unpackaged) file(s) found:<br>
>> 14:47:42 /usr/share/doc/ovirt-provider-<wbr>ovn/README.adoc<br>
>> 14:47:42 make: *** [rpm] Error 1<br>
>> 14:47:42 Took 2 seconds<br>
>><br>
>><br>
>> Possible root cause for that is [3] (Install README). However, I see CI<br>
>> passed there within the change at least before the rebase.<br>
>><br>
>><br>
>> [1]<br>
>> <a href="http://jenkins.ovirt.org/job/ovirt-master_change-queue-tester/1996/console" rel="noreferrer" target="_blank">http://jenkins.ovirt.org/job/<wbr>ovirt-master_change-queue-<wbr>tester/1996/console</a><br>
>> [2]<br>
>> <a href="http://jenkins.ovirt.org/job/ovirt-provider-ovn_master_build-artifacts-el7-x86_64/142/console" rel="noreferrer" target="_blank">http://jenkins.ovirt.org/job/<wbr>ovirt-provider-ovn_master_<wbr>build-artifacts-el7-x86_64/<wbr>142/console</a><br>
>> [3] <a href="https://gerrit.ovirt.org/#/c/80801/" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/<wbr>80801/</a><br>
>><br>
>> --<br>
>> Anton Marchukov<br>
>> Team Lead - Release Management - RHV DevOps - Red Hat<br>
<br>
</div></div>Thanks for your report, Anton. I'm sure that Marcin would fix this by tomorrow.<br>
<br>
However, we see here another case where a patch<br>
<a href="https://gerrit.ovirt.org/#/c/80801/" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/<wbr>80801/</a> triggers a job<br>
<a href="http://jenkins.ovirt.org/job/ovirt-provider-ovn_master_build-artifacts-el7-x86_64/134/" rel="noreferrer" target="_blank">http://jenkins.ovirt.org/job/<wbr>ovirt-provider-ovn_master_<wbr>build-artifacts-el7-x86_64/<wbr>134/</a><br>
the job fails, but the patch is not marked with -1, and the patch<br>
author does not get an email.<br>
<br>
I know that the CI team is paying attention to this glitch, but please<br>
note that it's here, too.<br>
<br>
Marcin, in vdsm's automation/check-patch we trigger built-artifacts in<br>
case a Makefile or a spec has changed. How about doing the same in<br>
ovirt-provifder-ovn?<br>
<br>
Regards,<br>
Dan.<br>
</blockquote></div><br></div>