<div dir="ltr">Hi Anita,<br><div class="gmail_extra">thanks for your answer.<br><br></div><div class="gmail_extra">Since we noticed that our CI posted comments with no logs we stopped commenting on patches.<br></div><div class="gmail_extra"><br></div><div class="gmail_extra">We were thinking to backport the patch mentioned by Jeremy in order to avoid waiting for the next zuul release.<br>The idea is to correctly test the feature on the openstack-dev/ci-sandbox before starting commenting on other projects.<br></div><div class="gmail_extra">What do you think?<br></div><div class="gmail_extra"><br></div><div class="gmail_extra">Best regards,<br></div>Francesco</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 21, 2015 at 4:04 PM, Anita Kuno <span dir="ltr"><<a href="mailto:anteaya@anteaya.info" target="_blank">anteaya@anteaya.info</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 10/21/2015 09:17 AM, Jeremy Stanley wrote:<br>
> On 2015-10-21 13:36:41 +0200 (+0200), Francesco Santoro wrote:<br>
> [...]<br>
>> This setup is correctly working but zuul publishes comments also on<br>
>> other patches that don't trigger any job.<br>
> [...]<br>
><br>
> It's because file and branch filters affect which jobs run, but<br>
> comments are tied to whether pipelines are triggered (not to<br>
> individual jobs).<br>
><br>
> What you probably want is <a href="https://review.openstack.org/188383" rel="noreferrer" target="_blank">https://review.openstack.org/188383</a> which<br>
> will cause Zuul not to report for a pipeline at all if it has no<br>
> matching jobs for a change triggering it. This merged to master a<br>
> few weeks after 2.1.0 was released so isn't in an officially<br>
> released version yet. You could try running from the tip of the<br>
> master branch to get this improvement, or backport the patch to your<br>
> installed version, or wait for the next release.<br>
><br>
<br>
</span>Please note that should you choose to wait for the next release, having<br>
your third party ci system post comments to gerrit that contain no logs<br>
is contrary to the requirements of third party ci systems:<br>
<a href="http://docs.openstack.org/infra/system-config/third_party.html#requirements" rel="noreferrer" target="_blank">http://docs.openstack.org/infra/system-config/third_party.html#requirements</a><br>
<br>
Jeremy was answering your technical inquiry about zuul and its versions.<br>
I am addressing the expectations of every third party ci operator. You<br>
are responsible for knowing and adhering to requirements.<br>
<br>
Thank you,<br>
Anita.<br>
<br>
_______________________________________________<br>
OpenStack-Infra mailing list<br>
<a href="mailto:OpenStack-Infra@lists.openstack.org">OpenStack-Infra@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra</a><br>
</blockquote></div><br></div>