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

sesheta (prow / commenter) auto-closes issues when it should not #2025

Closed
codificat opened this issue Oct 11, 2021 · 2 comments
Closed

sesheta (prow / commenter) auto-closes issues when it should not #2025

codificat opened this issue Oct 11, 2021 · 2 comments
Assignees
Labels
human_intervention_required kind/bug Categorizes issue or PR as related to a bug. triage/accepted Indicates an issue or PR is ready to be actively worked on.

Comments

@codificat
Copy link
Member

Describe the bug

We have seen the bots auto-close some issuses, supposedly due to lifecycle ("rotten"), when they did not meet the conditions.

Problems observed:

  • closing issues that have the lifecycle/frozen label (these should never be auto-closed)
  • closing issues at the same time that the lifecycle/rotten label is added (it should wait an additional 30 days before closing them)

To Reproduce
Wait enough time for issues to hit a lifecycle milestone (30 days, 720 hours)

Expected behavior
The lifecycle of the issues is properly tracked and managed by the bots using the appropriate labels.

Screenshots
Some examples of misbehaviour observed:

On the other hand, the bot has also been seen to act correctly, for example:

@codificat codificat added the kind/bug Categorizes issue or PR as related to a bug. label Oct 11, 2021
@sesheta sesheta added the needs-triage Indicates an issue or PR lacks a `triage/...` label and requires one. label Oct 11, 2021
@codificat
Copy link
Member Author

With the version bump in #2026 this seems to be fixed.

Looking at the logs and the history around when this was working / stopped working (early July), I believe that the cause of the problem was related to newline handling, fixed in kubernetes/test-infra#22922.

/close

@sesheta
Copy link
Member

sesheta commented Oct 15, 2021

@codificat: Closing this issue.

In response to this:

With the version bump in #2026 this seems to be fixed.

Looking at the logs and the history around when this was working / stopped working (early July), I believe that the cause of the problem was related to newline handling, fixed in kubernetes/test-infra#22922.

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@sesheta sesheta closed this as completed Oct 15, 2021
@harshad16 harshad16 added the triage/accepted Indicates an issue or PR is ready to be actively worked on. label Oct 25, 2021
@sesheta sesheta removed the needs-triage Indicates an issue or PR lacks a `triage/...` label and requires one. label Oct 25, 2021
@harshad16 harshad16 moved this to Completed in SIG-DevSecOps Sep 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
human_intervention_required kind/bug Categorizes issue or PR as related to a bug. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Projects
No open projects
Status: Done
Development

No branches or pull requests

3 participants