Skip to content

RATIS-2432. Close PRs after three months of inactivity#1373

Open
spacemonkd wants to merge 2 commits intoapache:masterfrom
spacemonkd:RATIS-2432
Open

RATIS-2432. Close PRs after three months of inactivity#1373
spacemonkd wants to merge 2 commits intoapache:masterfrom
spacemonkd:RATIS-2432

Conversation

@spacemonkd
Copy link

@spacemonkd spacemonkd commented Mar 11, 2026

What changes were proposed in this pull request?

Currently the RATIS PR queue has many pull requests which are more than a year old and are possibly not being worked on actively.
It would be good to run a cleanup job which marks such PRs as stale and automatically closes them.
We are setting the review cycle duration to 3months

  • If for 77 days there has been no updates (this is a random number which I selected as a good indicator close to 3months) then we mark the PR as stale
  • If after that the author still doesn't update within two weeks, then we close the PR.

Side-note: The workflow runs weekly on Sundays. This means the duration can be off by at-most ~7 days. So PRs can take at most 84days to be marked as stale and 21days at most to be closed. But since we are not getting too many changes this is an acceptable tradeoff

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/RATIS-2432

How was this patch tested?

N/A

@spacemonkd spacemonkd marked this pull request as ready for review March 11, 2026 09:18
@spacemonkd
Copy link
Author

@adoroszlai could you take a look?

Copy link
Contributor

@adoroszlai adoroszlai left a comment

Choose a reason for hiding this comment

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

Thank @spacemonkd for the patch. Let's see what others think about the specific config.

Copy link
Contributor

@OneSizeFitsQuorum OneSizeFitsQuorum left a comment

Choose a reason for hiding this comment

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

LGTM, wait for others opinion

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.

3 participants