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

configure stale bot #43537

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

configure stale bot #43537

wants to merge 1 commit into from

Conversation

ndeloof
Copy link
Contributor

@ndeloof ndeloof commented Apr 27, 2022

- What I did
Added configuration for stale bot so that old issues without recent activity get closed.

With 4k open issues, we hardly can offer efficient triage, so using stale bot can help focus on active topics - we did the same on docker/compose. Bot is configured to exclude feature requests so that we can keep them alive for future investigation, or close those after review by an explicit decision.

note: Will require an admin on moby organization to install https://github.com/apps/stale

- kind/feature
- kind/enhancement
# Label to use when marking an issue as stale
staleLabel: wontfix
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't seem correct, as the bot may close fixable bug issues.
Maybe staleLabel: stale or staleLabel: closed-due-to-inactivity?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

indeed, copy/pasted the reference doc, but should better adopt stale we use here

Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
@thaJeztah
Copy link
Member

Thanks! I'll try to have a look at the configuration of this (need to think if there's other labels to exclude; e.g., I recall we added a confirmed label, and help-wanted or exp/xxx labels (as in: yes, these need to be fixed, and are ok to pick up). Also, e.g. roadmap issues, which may be open for some time (but generally shouldn't be closed).

In general 👍 on adding this though!

Copy link
Member

@cpuguy83 cpuguy83 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

# Set to true to ignore issues with an assignee (defaults to false)
exemptAssignees: true
# Label to use when marking an issue as stale
staleLabel: stale
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be status/stale to match other labels, or do we think stale is fine? (I don't personally have a strong opinion either way)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants