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

Airshipbot appears to be reopening issues right after closing them #502

Closed
eak13 opened this issue Mar 26, 2021 · 8 comments
Closed

Airshipbot appears to be reopening issues right after closing them #502

eak13 opened this issue Mar 26, 2021 · 8 comments
Assignees
Labels
4-CI/CD Relates to issues with Zuul & gating bug Something isn't working priority/medium Default priority for items size m
Milestone

Comments

@eak13
Copy link

eak13 commented Mar 26, 2021

Describe the bug
Several recent issues have been closed upon code merge & then immediately reopened by the Airshipbot. Examples are #482, #485, #370 & #419. Initially thought it might be due to there being both the "Relates-To:" and "Closes:" tags in the commit message but #419 does not appear to have both tags. The other three examples do. Per comments below, does not appear to be commit label related.

Expected behavior
Closed issues stay closed upon closing patchset merging.

@eak13 eak13 added bug Something isn't working triage Needs evaluation by project members labels Mar 26, 2021
@raliev12
Copy link
Contributor

@ak3216 I've investigated this issue a little bit, and it seems it has no relations with airshipctl code and update-github-issues job. This bug reproduces as sequence of the following events:

  1. Once a commit merged into master, it triggers airshipbot to close this issue
  2. post and promote jobs are starting; as part of promote job we run update-github-issues.py script, which leaves a comment to issue (The Change that closes this issue was merged) and removes some labels from it, like wip, ready-for-review and so on. Removing labels from the issue most likely is not the cause why the issue reopens again (since reopening in this case should happen immediately).
  3. After some time, once post and promote completed (like 5-15 minutes), some "gerrit" activity (Issue reopened due to new activity on Gerrit) triggers airshipbot to reopen the issue.

I can't say for sure, but 90% this issue isn't related with airshipctl code and zuul jobs. Something should be checked in airshipbot configuration. I don't know who owns it and how it is configured inside, but seems the root cause is there.

@eak13
Copy link
Author

eak13 commented Mar 29, 2021

@raliev12, thanks for looking into this. @sirajyasin, can you please see if something is going on with the airshipbot? Thanks.

@jezogwza jezogwza added 4-CI/CD Relates to issues with Zuul & gating priority/medium Default priority for items and removed triage Needs evaluation by project members labels Apr 7, 2021
@jezogwza jezogwza added this to the v2.1 milestone Apr 7, 2021
@sirajyasin
Copy link
Contributor

@ak3216 , I can look into this issue. Could you please assign this issue to me.

@eak13
Copy link
Author

eak13 commented Apr 16, 2021

All yours @sirajyasin, thanks!

@airshipbot
Copy link

airshipbot commented Apr 16, 2021

Related Change #786694

Subject: Disable the redundant github issue update job
Link: https://review.opendev.org/c/airship/airshipctl/+/786694
Status: MERGED
Owner: Sirajudeen ([email protected])

Approvals

Code-Review
+2 Ian Howell
+1 Bijaya Sharma
+2 Matt McEuen
Verified
+2 Zuul
Workflow
+1 Matt McEuen

Last Updated: 2021-04-20 20:39:46 CDT

@eak13 eak13 added the size m label Apr 19, 2021
airshipbot pushed a commit that referenced this issue Apr 21, 2021
* airship-airshipctl-update-github-issues job adds comments
  to related github issue on PS merge based on the tag(Realtes-To/Closes)
  in commit message.
* airshipbot is already doing the job of adding comments to issues on new PSs
  and updating the status (closed, reopen) as applicable.
* So the job airship-airshipctl-update-github-issues is redundant when airshipbot
  is running.

Change-Id: I2c6fa748d3334384bf5b31c87d9501a006a153c1
Relates-To: #502
@airshipbot
Copy link

airshipbot commented Apr 21, 2021

Related Change #787250

Subject: Delete GitHub token
Link: https://review.opendev.org/c/airship/airshipctl/+/787250
Status: NEW
Owner: Roman Gorshunov ([email protected])

Approvals

Code-Review
+1 Sirajudeen
+1 Bijaya Sharma
+1 Sean Eagan
+2 Drew Walters
Verified
+1 Zuul
Workflow
! None

Last Updated: 2021-04-26 16:59:04 CDT

@sirajyasin
Copy link
Contributor

The Related PSs for this Fix are merged. Will observe the BOT behavior for few days before closing the issue.

@eak13
Copy link
Author

eak13 commented Apr 29, 2021

Closing this out as it appears to be resolved. No longer seeing issues being reopened.

@eak13 eak13 closed this as completed Apr 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4-CI/CD Relates to issues with Zuul & gating bug Something isn't working priority/medium Default priority for items size m
Projects
None yet
Development

No branches or pull requests

5 participants