[infrastructure] github-automation: pings missing in backport PRs

[kleisauke]

Looking at the GitHub automation routines script (i.e. the one that handles the /cherry-pick command), I think the last line of this code snippet is wrongly indented:

llvm-project/llvm/utils/git/github-automation.py

Lines 497 to 500 in c320df4

	for review in pull.get_reviews():
	if review.state != "APPROVED":
	continue
	reviewers.append(review.user.login)

i.e. that line should probably be part of the for loop:

--- a/llvm/utils/git/github-automation.py
+++ b/llvm/utils/git/github-automation.py
@@ -497,7 +497,7 @@ class ReleaseWorkflow:
                 for review in pull.get_reviews():
                     if review.state != "APPROVED":
                         continue
-                reviewers.append(review.user.login)
+                    reviewers.append(review.user.login)
         if len(reviewers):
             message = "{} What do you think about merging this PR to the release branch?".format(
                 " ".join(["@" + r for r in reviewers])

Noticed this while using this script in another project (see libvips/libvips#4162):

error: Failed while searching for reviewers local variable 'review' referenced before assignment

https://github.com/kleisauke/libvips/actions/runs/10957466259/job/30425668132#step:4:19

[llvmbot]

@llvm/issue-subscribers-infrastructure

Author: Kleis Auke Wolthuizen (kleisauke)

<!--IGNORE-->

Looking at the GitHub automation routines script (i.e. the one that handles the /cherry-pick command), I think the last line of this code snippet is wrongly indented:

llvm-project/llvm/utils/git/github-automation.py

Lines 497 to 500 in c320df4

	for review in pull.get_reviews():
	if review.state != "APPROVED":
	continue
	reviewers.append(review.user.login)

i.e. that line should probably be part of the for loop:

--- a/llvm/utils/git/github-automation.py
+++ b/llvm/utils/git/github-automation.py
@@ -497,7 +497,7 @@ class ReleaseWorkflow:
                 for review in pull.get_reviews():
                     if review.state != "APPROVED":
                         continue
-                reviewers.append(review.user.login)
+                    reviewers.append(review.user.login)
         if len(reviewers):
             message = "{} What do you think about merging this PR to the release branch?".format(
                 " ".join(["@" + r for r in reviewers])

Noticed this while using this script in another project (see libvips/libvips#4162):

error: Failed while searching for reviewers local variable 'review' referenced before assignment

https://github.com/kleisauke/libvips/actions/runs/10957466259/job/30425668132#step:4:19

[llvmbot]

Looking at the GitHub automation routines script (i.e. the one that handles the /cherry-pick command), I think the last line of this code snippet is wrongly indented:

llvm-project/llvm/utils/git/github-automation.py

Lines 497 to 500 in c320df4

	for review in pull.get_reviews():
	if review.state != "APPROVED":
	continue
	reviewers.append(review.user.login)

i.e. that line should probably be part of the for loop:

--- a/llvm/utils/git/github-automation.py
+++ b/llvm/utils/git/github-automation.py
@@ -497,7 +497,7 @@ class ReleaseWorkflow:
                for review in pull.get_reviews():
                    if review.state != "APPROVED":
                        continue
-                reviewers.append(review.user.login)
+                    reviewers.append(review.user.login)
        if len(reviewers):
            message = "{} What do you think about merging this PR to the release branch?".format(
                " ".join(["@" + r for r in reviewers])

Noticed this while using this script in another project (see libvips/libvips#4162):

error: Failed while searching for reviewers local variable 'review' referenced before assignment

https://github.com/kleisauke/libvips/actions/runs/10957466259/job/30425668132#step:4:19

Error: Command failed due to missing milestone.

[llvmbot]

@llvm/issue-subscribers-infrastructure

Author: Kleis Auke Wolthuizen (kleisauke)

<!--IGNORE-->

Looking at the GitHub automation routines script (i.e. the one that handles the /cherry-pick command), I think the last line of this code snippet is wrongly indented:

llvm-project/llvm/utils/git/github-automation.py

Lines 497 to 500 in c320df4

	for review in pull.get_reviews():
	if review.state != "APPROVED":
	continue
	reviewers.append(review.user.login)

i.e. that line should probably be part of the for loop:

--- a/llvm/utils/git/github-automation.py
+++ b/llvm/utils/git/github-automation.py
@@ -497,7 +497,7 @@ class ReleaseWorkflow:
                for review in pull.get_reviews():
                    if review.state != "APPROVED":
                        continue
-                reviewers.append(review.user.login)
+                    reviewers.append(review.user.login)
        if len(reviewers):
            message = "{} What do you think about merging this PR to the release branch?".format(
                " ".join(["@" + r for r in reviewers])

Noticed this while using this script in another project (see libvips/libvips#4162):

error: Failed while searching for reviewers local variable 'review' referenced before assignment

https://github.com/kleisauke/libvips/actions/runs/10957466259/job/30425668132#step:4:19

Error: Command failed due to missing milestone.

[kleisauke]

Triage: This issue is likely the root cause of sometimes missing pings, as noted in #107146 (comment) and #112258 (comment). /cc @mstorsjo FYI.

The "error" only occurs when a PR is merged without approval, which likely doesn't apply to LLVM. It's also properly caught here anyway:

llvm-project/llvm/utils/git/github-automation.py

Lines 590 to 593 in da9fece

	try:
	self.pr_request_review(pull)
	except Exception as e:
	print("error: Failed while searching for reviewers", e)

I've clarified the issue title accordingly.