<div dir="ltr">I just realized that the new syntax doesn't work when third party jobs use an old version of Zuul (e.g. RDO RCI).<div><br></div><div>Which means:</div><div>Depends-On: <a href="https://review.openstack.org/#/c/542556/">https://review.openstack.org/#/c/542556/</a></div><div>doesn't work</div><div><br></div><div>and</div><div>Depends-On: Ia30965b362d1c05d216f59b4cc1b3cb7e1284046</div><div>works for third party jobs.</div><div><br></div><div>We have to be very careful how we use the feature in TripleO CI. I've lost a bit of time trying to figuring out why my code wasn't passing our functional tests when I realized my code wasn't properly checkout.</div><div><br></div><div>My recommendation for TripleO devs: use the old syntax if you want your code to be tested by RDO Third party CI (now voting btw).</div><div><br></div><div>Thanks,</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 5, 2018 at 1:11 PM, Alex Schultz <span dir="ltr"><<a href="mailto:aschultz@redhat.com" target="_blank">aschultz@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 Thu, Feb 1, 2018 at 11:55 AM, James E. Blair <<a href="mailto:corvus@inaugust.com">corvus@inaugust.com</a>> wrote:<br>
> Zane Bitter <<a href="mailto:zbitter@redhat.com">zbitter@redhat.com</a>> writes:<br>
><br>
>> Yeah, it's definitely nice to have that flexibility. e.g. here is a<br>
>> patch that wouldn't merge for 3 months because the thing it was<br>
>> dependent on also got proposed as a backport:<br>
>><br>
>> <a href="https://review.openstack.org/#/c/514761/1" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/514761/1</a><br>
>><br>
>> From an OpenStack perspective, it would be nice if a Gerrit ID implied<br>
>> a change from the same Gerrit instance as the current repo and the<br>
>> same branch as the current patch if it exists (otherwise any branch),<br>
>> and we could optionally use a URL instead to select a particular<br>
>> change.<br>
><br>
> Yeah, that's reasonable, and it is similar to things Zuul does in other<br>
> areas, but I think one of the thing we want to do with Depends-On is<br>
> consider that Zuul isn't the only audience.  It's there just as much for<br>
> the reviewers, and other folks.  So when it comes to Gerrit change ids,<br>
> I feel we had to constrain it to Gerrit's own behavior.  When you click<br>
> on one of those in Gerrit, it shows you all of the changes across all of<br>
> the repos and branches with that change-id.  So that result list is what<br>
> Zuul should work with.  Otherwise there's a discontinuity between what a<br>
> user sees when they click the hyperlink under the change-id and what<br>
> Zuul does.<br>
><br>
> Similarly, in the new system, you click the URL and you see what Zuul is<br>
> going to use.<br>
><br>
> And that leads into the reason we want to drop the old syntax: to make<br>
> it seamless for a GitHub user to know how to Depends-On a Gerrit change,<br>
> and vice versa, with neither requiring domain-specific knowledge about<br>
> the system.<br>
><br>
<br>
</div></div>While I can appreciate that, having to manage urls for backports in<br>
commit messages will lead to missing patches and other PEBAC related<br>
problems. Perhaps rather than throwing out this functionality we can<br>
push for improvements in the gerrit interaction itself?  I'm really -1<br>
on removing the change-id syntax just for this reasoning. The UX of<br>
having to manage complex depends-on urls for things like backports<br>
makes switching to URLs a non-starter unless I have a bunch of<br>
external system deps (and I generally don't).<br>
<br>
Thanks,<br>
-Alex<br>
<div class="HOEnZb"><div class="h5"><br>
> -Jim<br>
><br>
> ______________________________<wbr>______________________________<wbr>______________<br>
> OpenStack Development Mailing List (not for usage questions)<br>
> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
<br>
______________________________<wbr>______________________________<wbr>______________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Emilien Macchi<br></div></div>
</div>