From f370ca6621f69a1969c1d0aeb5b6ee6b1cfa3186 Mon Sep 17 00:00:00 2001 From: Ramy Asselin Date: Mon, 18 Jul 2016 04:48:11 -0700 Subject: [PATCH] Stop Jenkin master restarts Jenkin masters are no longer in use. Stop restarting them. Change-Id: Idc66665e341f2be36f3ec269a535333cb5db868c --- manifests/site.pp | 2 -- modules/openstack_project/manifests/puppetmaster.pp | 10 ++++++---- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/manifests/site.pp b/manifests/site.pp index 9c43517b5f..e856957d50 100644 --- a/manifests/site.pp +++ b/manifests/site.pp @@ -180,8 +180,6 @@ node 'puppetmaster.openstack.org' { } class { 'openstack_project::puppetmaster': root_rsa_key => hiera('puppetmaster_root_rsa_key'), - jenkins_api_user => hiera('jenkins_api_user', 'username'), - jenkins_api_key => hiera('jenkins_api_key'), puppetmaster_clouds => hiera('puppetmaster_clouds'), } } diff --git a/modules/openstack_project/manifests/puppetmaster.pp b/modules/openstack_project/manifests/puppetmaster.pp index 82c9a6a2fc..2e80f96cc6 100644 --- a/modules/openstack_project/manifests/puppetmaster.pp +++ b/modules/openstack_project/manifests/puppetmaster.pp @@ -1,9 +1,7 @@ # == Class: openstack_project::puppetmaster # class openstack_project::puppetmaster ( - $jenkins_api_key, $puppetmaster_clouds, - $jenkins_api_user = 'hudson-openstack', $root_rsa_key = 'xxx', $puppetdb = true, $puppetdb_server = 'puppetdb.openstack.org', @@ -219,7 +217,7 @@ class openstack_project::puppetmaster ( # Jenkins master management cron { 'restartjenkinsmasters': - user => 'root', + ensure => absent, # Run through all masters onces a week. weekday => '6', hour => '0', @@ -228,8 +226,12 @@ class openstack_project::puppetmaster ( command => "flock -n /var/run/puppet/restart_jenkins_masters.lock ansible-playbook -f 1 /opt/system-config/production/playbooks/restart_jenkins_masters.yaml --extra-vars 'user=${jenkins_api_user} password=${jenkins_api_key}' >> /var/log/restart_jenkins_masters.log 2>&1", } + file { '/var/log/restart_jenkins_masters.log': + ensure => absent, + } + logrotate::file { 'restartjenkinsmasters': - ensure => present, + ensure => absent, log => '/var/log/restart_jenkins_masters.log', options => ['compress', 'copytruncate',