Dzahn has submitted this change and it was merged.
Change subject: Revert "contint: tidy Nodepool slaves config history"
......................................................................
Revert "contint: tidy Nodepool slaves config history"
Puppet tidy lacks a bunch of features such as being able
to do a match against the whole patch. It does it solely
against the file basename which is not what I wanted to do.
This reverts commit 4bbfd3b9ec21e044a15d0c4e78736cac2332af2c.
Bug: T126552
Change-Id: Ibf289e4f6cd77a3367f7b5ebe4a9283749b9338c
---
M modules/role/manifests/ci/master.pp
1 file changed, 0 insertions(+), 9 deletions(-)
Approvals:
jenkins-bot: Verified
Dzahn: Looks good to me, approved
diff --git a/modules/role/manifests/ci/master.pp
b/modules/role/manifests/ci/master.pp
index 921b074..18979ab 100644
--- a/modules/role/manifests/ci/master.pp
+++ b/modules/role/manifests/ci/master.pp
@@ -21,15 +21,6 @@
backup::set { 'contint' : }
- # Nodepool spawn non ephemeral slaves which causes config-history plugin to
- # fill up entries until it reaches the limit of 32k inodes. T126552
- tidy { 'history of nodepool slaves config':
- path => '/var/lib/jenkins/config-history/nodes',
- age => '1d',
- recurse => true,
- matches => 'ci-*_deleted_*',
- }
-
nrpe::monitor_service { 'jenkins_zmq_publisher':
description => 'jenkins_zmq_publisher',
contact_group => 'contint',
--
To view, visit https://gerrit.wikimedia.org/r/300085
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ibf289e4f6cd77a3367f7b5ebe4a9283749b9338c
Gerrit-PatchSet: 4
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Hashar <[email protected]>
Gerrit-Reviewer: Dzahn <[email protected]>
Gerrit-Reviewer: Hashar <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
