<div dir="ltr"><div>To avoid zuul (v2.1.0) posting comments with no logs we need to backport the patch mentioned by Jeremy (<a href="https://review.openstack.org/188383" rel="noreferrer" target="_blank">https://review.openstack.org/188383</a><span></span>).<br><br>Clone zuul project and cherry-pick the mentioned patch:<br><br>- git clone <a href="https://review.openstack.org/openstack-infra/zuul">https://review.openstack.org/openstack-infra/zuul</a><br></div><div>- cd zuul && git checkout tags/2.1.0<br>- git fetch <a href="https://review.openstack.org/openstack-infra/zuul">https://review.openstack.org/openstack-infra/zuul</a> refs/changes/83/188383/3 && git cherry-pick FETCH_HEAD<br><br></div><div><br>Install patched zuul:<br></div><div>- python setup.py install<br></div><div><br></div><div><br>In zuul/layout.yaml file defines a filter for a job to be run on openstack-dev/ci-sandbox patchset creation<br><br>jobs:<br>  - name: test-job<br>    files:<br>      - '^test-1$'<br>    voting: true<br><br></div><div>Run zuul-server as usual<br></div><div>Upload a patch for review modifying test-1 and verify that CI correctly posts a comment with link to logs<br></div><div>Upload a patch for review modifying test-2 and verify that CI does not post any comment<br><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 21, 2015 at 5:02 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 10:58 AM, Francesco Santoro wrote:<br>
> Hi Anita,<br>
> thanks for your answer.<br>
><br>
> Since we noticed that our CI posted comments with no logs we stopped<br>
> commenting on patches.<br>
<br>
</span>Oh how heartening, thank you. Your level of responsibility regarding<br>
your third party ci's effect on the community makes me happy for the<br>
day. Thank you, Francesco!<br>
<span class=""><br>
><br>
> We were thinking to backport the patch mentioned by Jeremy in order to<br>
> avoid waiting for the next zuul release.<br>
<br>
</span>That certainly is an option, yes.<br>
<span class=""><br>
> The idea is to correctly test the feature on the openstack-dev/ci-sandbox<br>
> before starting commenting on other projects.<br>
> What do you think?<br>
<br>
</span>Oh my goodness, two wonderful things you acknowledge in your email.<br>
Thank you so much. I look forward to meeting you some day and shaking<br>
your hand. Thank you Francesco. Yes, the ci-sandbox repo is the place to<br>
test third party ci operations. My irc nickname is anteaya, if you want<br>
me to look at any of your output while you are testing do ping me in<br>
#openstack-infra.<br>
<br>
Thank you so much, Francesco,<br>
Anita.<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
> Best regards,<br>
> Francesco<br>
><br>
> On Wed, Oct 21, 2015 at 4:04 PM, Anita Kuno <<a href="mailto:anteaya@anteaya.info">anteaya@anteaya.info</a>> wrote:<br>
><br>
>> 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>
>> 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>
>><br>
><br>
<br>
</div></div></blockquote></div><br></div>