Reduce runner stale timeout from 3 months to 7 days
What does this MR do and why?
This MR reduces runner stale timeout from 3 months to 7 days. This causes more runners to classify as stale, and to be cleaned up by the Ci::Runners::StaleGroupRunnersPruneCronWorker
Changelog: changed
Closes #466312 (closed)
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
Before | After |
---|---|
![]() |
![]() |
How to set up and validate locally
Numbered steps to set up and validate the change are strongly suggested.
n/a
Merge request reports
Activity
changed milestone to %17.1
added Category:Fleet Visibility devopsverify grouprunner sectionci labels
assigned to @pedropombeiro
added pipelinetier-1 label
- A deleted user
added backend label
2 Warnings This MR changes code in ee/
, but its Changelog commit is missing theEE: true
trailer. Consider adding it to your Changelog commits.There were no new or modified feature flag YAML files detected in this MR. If the changes here are already controlled under an existing feature flag, please add
the feature flagexists. Otherwise, if you think the changes here don't need
to be under a feature flag, please add the label feature flagskipped, and
add a short comment about why we skipped the feature flag.For guidance on when to use a feature flag, please see the documentation.
1 Message This merge request adds or changes documentation files. A review from the Technical Writing team before you merge is recommended. Reviews can happen after you merge. Documentation review
The following files require a review from a technical writer:
-
doc/administration/instance_limits.md
(Link to current live version) -
doc/api/graphql/reference/index.md
(Link to current live version) -
doc/ci/runners/runners_scope.md
(Link to current live version)
The review does not need to block merging this merge request. See the:
-
Metadata for the
*.md
files that you've changed. The first few lines of each*.md
file identify the stage and group most closely associated with your docs change. - The Technical Writer assigned for that stage and group.
- Documentation workflows for information on when to assign a merge request for review.
Reviewer roulette
Category Reviewer Maintainer backend @ibaum
(UTC-5, 7 hours behind author)
@wandering_person
(UTC+7, 5 hours ahead of author)
test for spec/features/*
@ibaum
(UTC-5, 7 hours behind author)
Maintainer review is optional for test for spec/features/*
~"Verify" Reviewer review is optional for ~"Verify" @drew
(UTC+0, 2 hours behind author)
Please check reviewer's status!
Please refer to documentation page for guidance on how you can benefit from the Reviewer Roulette, or use the GitLab Review Workload Dashboard to find other available reviewers.
If needed, you can retry the
danger-review
job that generated this comment.Generated by
Danger-
removed backend pipelinetier-1 labels
added pipelinetier-1 label
added featureenhancement typefeature labels
removed typefeature label
removed featureenhancement label
- Resolved by Leaminn Ma
requested review from @aluthra2
- Resolved by 🤖 GitLab Bot 🤖
Proper labels assigned to this merge request. Please ignore me.
@pedropombeiro
- please see the following guidance and update this merge request.1 Error Please add typebug typefeature, or typemaintenance label to this merge request. Edited by 🤖 GitLab Bot 🤖
added 1 commit
- d5d11d25 - Reduce runner stale timeout from 3 months to 7 days
- A deleted user
added backend label
added 1 commit
- 64abee95 - Reduce runner stale timeout from 3 months to 7 days
added 1 commit
- df6b520a - Reduce runner stale timeout from 3 months to 7 days
- A deleted user
added documentation label
added 1 commit
- b1e6a8a6 - Reduce runner stale timeout from 3 months to 7 days
added 1 commit
- 4fa1fcad - Reduce runner stale timeout from 3 months to 7 days
added pipeline:mr-approved label
- Resolved by Pedro Pombeiro (OOO from Feb 17th-21st)
@aluthra2
, thanks for approving this merge request.This is the first time the merge request has been approved. To ensure we don't only run predictive pipelines, and we don't break
master
, a new pipeline will be started shortly.Please wait for the pipeline to start before resolving this discussion and set auto-merge for the new pipeline. See merging a merge request for more details.
added pipelinetier-2 label and removed pipelinetier-1 label
requested review from @wandering_person and removed review request for @aluthra2
E2E Test Result Summary
allure-report-publisher
generated test report!e2e-test-on-gdk:
test report for ab4b900aexpand test summary
+------------------------------------------------------------------+ | suites summary | +-------------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +-------------+--------+--------+---------+-------+-------+--------+ | Manage | 0 | 0 | 1 | 0 | 1 | ➖ | | Plan | 56 | 0 | 2 | 0 | 58 | ✅ | | Create | 122 | 0 | 10 | 0 | 132 | ✅ | | Verify | 30 | 0 | 2 | 0 | 32 | ✅ | | Govern | 64 | 0 | 1 | 0 | 65 | ✅ | | Data Stores | 31 | 0 | 0 | 0 | 31 | ✅ | | Package | 19 | 0 | 12 | 0 | 31 | ✅ | | Monitor | 8 | 0 | 0 | 0 | 8 | ✅ | | Fulfillment | 1 | 0 | 0 | 0 | 1 | ✅ | | Release | 5 | 0 | 0 | 0 | 5 | ✅ | | Analytics | 2 | 0 | 0 | 0 | 2 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+ | Total | 338 | 0 | 28 | 0 | 366 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+
e2e-package-and-test:
test report for ab4b900aexpand test summary
+-------------------------------------------------------------+ | suites summary | +--------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +--------+--------+--------+---------+-------+-------+--------+ | Create | 276 | 0 | 30 | 6 | 306 | ✅ | | Verify | 49 | 0 | 11 | 0 | 60 | ✅ | | Plan | 4 | 0 | 0 | 0 | 4 | ✅ | | Govern | 2 | 0 | 0 | 0 | 2 | ✅ | +--------+--------+--------+---------+-------+-------+--------+ | Total | 331 | 0 | 41 | 6 | 372 | ✅ | +--------+--------+--------+---------+-------+-------+--------+
added featureenhancement label
added typefeature label
added 41 commits
-
4fa1fcad...2d7dba10 - 39 commits from branch
master
- 46bea56e - Reduce runner stale timeout from 3 months to 7 days
- df2b98c5 - Fix missed specs
-
4fa1fcad...2d7dba10 - 39 commits from branch
added featureconsolidation label and removed featureenhancement label
added 58 commits
-
df2b98c5...986e7c28 - 56 commits from branch
master
- 3e2cbf93 - Reduce runner stale timeout from 3 months to 7 days
- 014674cd - Fix missed specs
-
df2b98c5...986e7c28 - 56 commits from branch
- Resolved by Ashraf Khamis
- Resolved by Pedro Pombeiro (OOO from Feb 17th-21st)
- Resolved by Pedro Pombeiro (OOO from Feb 17th-21st)
added Technical Writing label
requested review from @ashrafkhamis
added 15 commits
-
014674cd...da628f87 - 12 commits from branch
master
- 3dcf874e - Reduce runner stale timeout from 3 months to 7 days
- 6fe6d701 - Fix missed specs
- dcdc3807 - Add history entry to documentation
Toggle commit list-
014674cd...da628f87 - 12 commits from branch
requested review from @lma-git
reset approvals from @wandering_person by pushing to the branch
- Resolved by Pedro Pombeiro (OOO from Feb 17th-21st)
removed review request for @ashrafkhamis
requested review from @ashrafkhamis
added docsfeature label
removed review request for @ashrafkhamis
mentioned in merge request !155885 (merged)
added pipelinetier-3 label and removed pipelinetier-2 label
enabled automatic add to merge train when the pipeline for c096b3c3 succeeds
removed this merge request from the merge train because pipeline did not succeed. Learn more.
enabled automatic add to merge train when the pipeline for aafd16cc succeeds
mentioned in commit b28ca974
added workflowstaging-canary label
added workflowcanary label and removed workflowstaging-canary label
added workflowstaging label and removed workflowcanary label
added workflowproduction label and removed workflowstaging label
mentioned in merge request !156194 (merged)
added releasedcandidate label
added releasedpublished label and removed releasedcandidate label
mentioned in merge request kubitus-project/kubitus-installer!3098 (merged)