Hashar has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/300085

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(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/85/300085/1

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: newchange
Gerrit-Change-Id: Ibf289e4f6cd77a3367f7b5ebe4a9283749b9338c
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Hashar <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to