<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jun 14, 2022 at 9:04 PM Jeremy Stanley <<a href="mailto:fungi@yuggoth.org">fungi@yuggoth.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 2022-06-14 20:40:30 +0900 (+0900), Takashi Kajinami wrote:<br>
[...]<br>
> In the past, we abandoned backport without checking this and this<br>
> resulted in broken commit id in some backports. (stable/train<br>
> backports point to the commit id of ussuri backport, which was<br>
> never merged).<br>
[...]<br>
<br>
Not sure if it's too heavy-handed, but one workaround would be to<br>
add a Depends-On footer in the commit message of each backport<br>
referencing the change for the branch ahead of it. Then, Zuul would<br>
only ever allow the backport to merge if the same change on all<br>
newer branches had already merged successfully.<br></blockquote><div>Yeah I think that would be a solution though it requires relatively big overhead,</div><div>because we need to manually add that Depends-on line.<br></div><div><br></div><div>I sometimes hope zuul can block backports depending on the "cherry picked from commit ..." line</div><div>but it might be worth discussing as a potential enhancement with the zuul devs.<br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
-- <br>
Jeremy Stanley<br>
</blockquote></div></div>