Skip to content
This repository has been archived by the owner on May 20, 2024. It is now read-only.

Bot removing ci_verified from old PRs #1188

Closed
mattclay opened this issue Apr 17, 2019 · 6 comments
Closed

Bot removing ci_verified from old PRs #1188

mattclay opened this issue Apr 17, 2019 · 6 comments
Labels

Comments

@mattclay
Copy link
Member

The bot is removing the ci_verified label from old PRs which have not seen any recent updates.

The most recent example is this PR: ansible/ansible#38841

However, I've seen much older PRs updated as well: ansible/ansible#24111

The last CI run from that PR was almost 2 years ago.

@mattclay mattclay added the bug label Apr 17, 2019
@mattclay
Copy link
Member Author

https://app.shippable.com/runs/5903455c50821b07007ef4e5 is the latest CI run for ansible/ansible#24111 -- I was able to download all the artifacts and console logs for this run, and the data appears to be intact.

@jctanner
Copy link
Contributor

Apparently I'm already handling this in the most logical way possible: https://github.com/ansible/ansibullbot/blob/master/ansibullbot/triagers/plugins/needs_revision.py#L578-L590

I'll have to figure out a better way to track whatever is going on in the prod instance.

@jctanner
Copy link
Contributor

I checked both issues mentioned above locally and both fall into the same code path in my previous comment and ci_verified gets flipped back to True.

@mkrizek
Copy link
Collaborator

mkrizek commented Aug 6, 2020

@mattclay @samdoran given that this issue is more than year old, is this still an issue?

@mattclay
Copy link
Member Author

mattclay commented Aug 6, 2020

I haven't noticed this in a while, although it doesn't mean it isn't occurring. I'm fine closing this.

@mkrizek
Copy link
Collaborator

mkrizek commented Aug 7, 2020

If you see this happening again please open a new issue or re-open this one, thanks!

@mkrizek mkrizek closed this as completed Aug 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants