<div dir="ltr">The PR processor has been merged/enabled. There was a bug where there might have been some dupe comments added to issues and PRs. Apologies.<br clear="all"><div><div dir="ltr" class="gmail_signature" 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><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 24, 2020 at 8:50 AM 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">After looking at some different options last night, I think a cron job is the only option. I opened this PR which has a scheduled job that should check for new PRs every 5 min:<div><br></div><div><a href="https://github.com/pulp/pulp-ci/pull/701" target="_blank">https://github.com/pulp/pulp-ci/pull/701</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><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 23, 2020 at 4:27 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">I ran into a problem. Any PR event in Github Actions runs against the fork and not the repo against which the PR was opened. And the repo's secrets such as the github api key and redmine api key are not shared with the fork for obvious security reasons. Fabricio and I are looking at some options. I'll try to figure out a workaround this coming weekend.<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><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 23, 2020 at 9:19 AM 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">Cool. When we move the cherry pick processor to Github Actions, we can do some cool things like run it when a PR gets merged or labeled.<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><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 23, 2020 at 9:08 AM Brian Bouterse <<a href="mailto:bmbouter@redhat.com" target="_blank">bmbouter@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>Oh I'm glad you asked. I incorrectly thought this also actually performed cherry picking, but I see now it only adds labels. At one point we had the idea to convert the PR processor to a "run as needed with github actions" instead of "run once a day with Travis".</div><div><br></div><div>I was referring to removing the cherry picking tooling from Travis, but if this doesn't replace it, let's not do that.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 23, 2020 at 9:05 AM 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">Brian,<div><br></div><div>What do you mean by "removing the old one"? AFAIK, there is nothing currently processing PRs and updating redmine.<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><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 23, 2020 at 8:41 AM Brian Bouterse <<a href="mailto:bmbouter@redhat.com" target="_blank">bmbouter@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>This is excellent. +1 to merging, rolling out to plugins, and removing the old one.</div><div><br></div><div>Thank you!<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 23, 2020 at 7:48 AM Ina Panova <<a href="mailto:ipanova@redhat.com" target="_blank">ipanova@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">Thank you for this work!<br clear="all"><div><div><div dir="ltr"><div dir="ltr"><div><div dir="ltr"><br><br>--------<br>Regards,<br><br>Ina Panova<br>Senior Software Engineer| Pulp| Red Hat Inc.<br><br>"Do not go where the path may lead,<br> go instead where there is no path and leave a trail."<br></div></div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Mar 21, 2020 at 3:49 PM Daniel Alley <<a href="mailto:dalley@redhat.com" target="_blank">dalley@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>This is great!   Awesome work, David <3</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Mar 21, 2020 at 10:38 AM 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">I created a PR workflow in Github Actions so I could learn more about Github Actions. <div><br></div><div>Here's the issue and my PR against plugin_template:<div><br></div><div><a href="https://pulp.plan.io/issues/4365" target="_blank">https://pulp.plan.io/issues/4365</a></div><div><a href="https://github.com/pulp/plugin_template/pull/199" target="_blank">https://github.com/pulp/plugin_template/pull/199</a><br><div><br></div><div>The PR processor will listen for new PRs in Github and when it detects a newly opened PR it does two things:</div><div><br></div><div>- Updates the <a href="http://pulp.plan.io" target="_blank">pulp.plan.io</a> issue</div><div>  - Adds a comment with a link to the Github PR (regardless of the issue's status)</div><div></div><div>  - Sets the issue status to POST if it's in NEW or ASSIGNED</div><div>- Adds labels to the Github PR<br></div><div>  - Right now, this is only "Needs Cherry Pick" which is added if tracker == "Issue"</div><div><br></div><div>I have an example that demonstrates the PR processor. Here's a story and its PR:</div><div><br></div><div><a href="https://pulp.plan.io/issues/6381" target="_blank">https://pulp.plan.io/issues/6381</a></div><div><a href="https://github.com/daviddavis/pulp_file/pull/11" target="_blank">https://github.com/daviddavis/pulp_file/pull/11</a></div><div><br></div><div>The PR processor will also comment with a warning if there is no issue attached: </div><div><br></div><div><a href="https://github.com/daviddavis/pulp_file/pull/12" target="_blank">https://github.com/daviddavis/pulp_file/pull/12</a><br></div><div><br></div><div>Questions and feedback welcome.</div><div><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></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>
_______________________________________________<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>
_______________________________________________<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>
</blockquote></div>
</blockquote></div>
</blockquote></div>
</blockquote></div>
</blockquote></div>