<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">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>