<div dir="ltr">I think it would be easy to automatically apply the cherry pick label with Github Actions. Github Actions has all sorts of events that can trigger workflows including opening a new PR[0]. This workflow could also automatically move the redmine issue to POST and comment with the PR too.<div><br></div><div>[0] <a href="https://help.github.com/en/actions/reference/events-that-trigger-workflows#pull-request-event-pull_request" target="_blank">https://help.github.com/en/actions/reference/events-that-trigger-workflows#pull-request-event-pull_request</a><br clear="all"><div><div dir="ltr" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><br></div><div>David</div></div></div></div></div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 18, 2020 at 6:02 AM Tatiana Tereshchenko <<a href="mailto:ttereshc@redhat.com" target="_blank">ttereshc@redhat.com</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"><div dir="ltr">I believe it's the responsibility of both the author and the PR reviewer.<div><br></div><div>If it's a one-time contribution from someone, then the PR reviewer is likely the one who is aware whether the cherry-pick should be done or not.</div><div>However in the majority of cases, we have regular contributors and they are aware of the process. Depending on the fix, they might be in a better position to say whether it's worth cherry-picking, if the cherry-pick will be clean or not.</div><div><br></div><div>Alternatively, can we automate it? When PR is open, look at the referred redmine issue and check its tracker, if it's an "issue", mark PR as the one to be cherry-picked, so reviewer can unset it if it's undesirable for whatever reason.</div><div><br></div><div>Tanya</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 17, 2020 at 9:43 PM David Davis <<a href="mailto:daviddavis@redhat.com" target="_blank">daviddavis@redhat.com</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"><div dir="ltr"><div>Today we missed a change that could have maybe have gone out with the 3.2 release. It stemmed from a lack of clarity around whose responsibility it is to label PRs with the cherry pick label. I think the general agreement is that it's ultimately the responsibility of the PR reviewer to add this label to the PR. I'm interested to see if there are any other thoughts or objections. Here is a PR I've opened as a proposal:</div><div><br></div><div><a href="https://github.com/pulp/pulpcore/pull/592" target="_blank">https://github.com/pulp/pulpcore/pull/592</a><br clear="all"><div><div dir="ltr"><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><br></div><div>David</div></div></div></div></div></div></div></div></div></div>
_______________________________________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a><br>
</blockquote></div>
</blockquote></div>