Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

contrib: Tighten search for list of PRs #12703

Merged

Conversation

christarazi
Copy link
Member

@christarazi christarazi commented Jul 29, 2020

Previously, if "set-labels.py" was in the PR title, then the grep
would pick up extraneous lines which throw off the parsing. See failed
example below:

$ ./contrib/backporting/submit-backport v1.8
...
Updating labels for PRs  * #12640 -- backporting: Report progress in set-labels.py (@pchaigno)
12640 12626 12632 12654 12651 12652 12659 12521 12683

Set labels for all PRs above? [y/N] y
usage: set-labels.py [-h] pr_number {pending,done} [version]
set-labels.py: error: argument pr_number: invalid int value: 'api'

Signal ERR caught!

Fixes: 3c4d43a ("contrib: Fix submit-backport PR set-labels detection")

@christarazi christarazi requested a review from a team as a code owner July 29, 2020 04:42
@christarazi christarazi added needs-backport/1.6 release-note/misc This PR makes changes that have no direct user impact. labels Jul 29, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot added dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Jul 29, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.8.3 Jul 29, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.7.7 Jul 29, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.6.11 Jul 29, 2020
Previously, if "set-labels.py" was in the PR title, then the `grep`
would pick up extraneous lines which throw off the parsing. See failed
example below:

```
$ ./contrib/backporting/submit-backport v1.8
...
Updating labels for PRs  * cilium#12640 -- backporting: Report progress in set-labels.py (@pchaigno)
12640 12626 12632 12654 12651 12652 12659 12521 12683

Set labels for all PRs above? [y/N] y
usage: set-labels.py [-h] pr_number {pending,done} [version]
set-labels.py: error: argument pr_number: invalid int value: 'api'

Signal ERR caught!
```

Fixes: 3c4d43a ("contrib: Fix submit-backport PR set-labels detection")

Signed-off-by: Chris Tarazi <chris@isovalent.com>
@christarazi christarazi force-pushed the pr/christarazi/fix-submit-backporting-script branch from 23e518e to 24584b1 Compare July 29, 2020 04:44
@christarazi christarazi added area/misc Impacts miscellaneous areas of the code not otherwise owned by another area. and removed area/area labels Jul 29, 2020
@coveralls
Copy link

coveralls commented Jul 29, 2020

Coverage Status

Coverage decreased (-0.008%) to 37.223% when pulling 24584b1 on christarazi:pr/christarazi/fix-submit-backporting-script into 4ea0ea4 on cilium:master.

@aanm aanm merged commit 1797d0e into cilium:master Jul 29, 2020
@christarazi christarazi deleted the pr/christarazi/fix-submit-backporting-script branch July 29, 2020 18:51
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.8 in 1.8.3 Jul 30, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.7 in 1.7.7 Jul 30, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.6 in 1.6.11 Jul 30, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.8 to Backport done to v1.8 in 1.8.3 Aug 3, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.7 to Backport done to v1.7 in 1.7.7 Aug 3, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.6 to Backport done to v1.6 in 1.6.11 Aug 3, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.6 to Backport done to v1.6 in 1.6.11 Aug 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/misc Impacts miscellaneous areas of the code not otherwise owned by another area. release-note/misc This PR makes changes that have no direct user impact.
Projects
No open projects
1.6.11
Backport done to v1.6
1.7.7
Backport done to v1.7
1.8.3
Backport done to v1.8
Development

Successfully merging this pull request may close these issues.

None yet

6 participants