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

Enable live runs of repo maintenence job via manual trigger #1300

Merged
merged 4 commits into from
Apr 3, 2024

Conversation

javulticat
Copy link
Member

@javulticat javulticat commented Feb 3, 2024

Description

Following up on the new automated repo maintenance job added in #1296 to help with handling ancient/inactive/abandoned issues/PRs, it has been successfully completing dry-runs as expected for a while now - it has run without issue since #1298 (which fixed a typo that was causing the job to fail) was merged ~2 weeks ago, and it's currently at ~150 consecutive successful runs (and counting).

This PR represents the next step toward enabling this bit of repo maintenance automation: dry-runs are being disabled, and live runs will begin. However, to err on the side of caution, this PR is also disabling the job's schedule which causes GitHub to automatically run it every 2 hours. So, just in case a live run causes an unexpected problem, the job won't keep running every 2 hours until a PR is able to be merged.

Since the job lacks automated scheduling for now, I will manually execute the job and verify each of its runs to make sure the job worked as expected before allowing it to run again. Once the job has successfully completed enough live runs that we can be highly confident that it is consistently performing as expected, the final step will be to re-enable the schedule for this job in a subsequent PR, allowing it to run automatically once again.

Blocked by

@javulticat javulticat added needs-review Needs attention from a maintainer repo-maintenance Will help maintain the repo labels Feb 3, 2024
@javulticat javulticat requested review from hellno and monkut February 3, 2024 00:52
@javulticat javulticat self-assigned this Feb 3, 2024
@javulticat javulticat enabled auto-merge (squash) February 3, 2024 00:52
@javulticat javulticat added the blocked Progress is blocked until another issue is resolved label Feb 3, 2024
@javulticat javulticat removed the blocked Progress is blocked until another issue is resolved label Feb 6, 2024
@coveralls
Copy link

coveralls commented Feb 6, 2024

Coverage Status

coverage: 74.831%. remained the same
when pulling 136b9c9 on jav/cleanup-inactive
into 4a24cb6 on master.

@javulticat javulticat disabled auto-merge April 3, 2024 13:43
@javulticat javulticat merged commit 047c9a8 into master Apr 3, 2024
6 checks passed
@javulticat javulticat deleted the jav/cleanup-inactive branch April 3, 2024 13:44
@javulticat
Copy link
Member Author

Decided to merge this as it had been stalled. I've since tested and confirmed that this has had the expected effects:

  • Manual runs can be triggered as expected
    image

  • Manually triggered dry-run ran successfully
    image

  • Manually triggered live run ran successfully...
    image

    • ...which left comments/labels as expected, starting from the oldest open issues/PRs
      image

@javulticat javulticat removed the needs-review Needs attention from a maintainer label Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
repo-maintenance Will help maintain the repo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants