From 33c0756ff825decfeeaad60d8435d90bb45d776a Mon Sep 17 00:00:00 2001
From: Monty Taylor <mordred@inaugust.com>
Date: Sat, 11 Aug 2018 07:23:27 -0500
Subject: [PATCH] Stop running puppet from puppetmaster

Bridge can run puppet on the remote hosts. Stop running on puppetmaster
so that we can run from bridge. Put it in the disabled group so that we
don't try to run puppet on it from bridge.

Change-Id: Ibcfa7e902c07c55e3a84f8232a11792c5f7d80e9
---
 .../manifests/puppetmaster.pp                 | 36 +++----------------
 .../roles/install-ansible/files/groups.yaml   |  2 +-
 2 files changed, 6 insertions(+), 32 deletions(-)

diff --git a/modules/openstack_project/manifests/puppetmaster.pp b/modules/openstack_project/manifests/puppetmaster.pp
index 10cb964b88..7f135973c5 100644
--- a/modules/openstack_project/manifests/puppetmaster.pp
+++ b/modules/openstack_project/manifests/puppetmaster.pp
@@ -33,14 +33,7 @@ class openstack_project::puppetmaster (
   }
 
   cron { 'updatecloudlauncher':
-    user        => 'root',
-    minute      => '0',
-    hour        => '*/1',
-    monthday    => '*',
-    month       => '*',
-    weekday     => '*',
-    command     => 'flock -n /var/run/puppet/puppet_run_cloud_launcher.lock bash /opt/system-config/production/run_cloud_launcher.sh >> /var/log/puppet_run_cloud_launcher_cron.log 2>&1',
-    environment => 'PATH=/var/lib/gems/1.8/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin',
+    ensure => absent,
   }
 
   logrotate::file { 'updatecloudlauncher':
@@ -72,14 +65,7 @@ class openstack_project::puppetmaster (
   }
 
   cron { 'updatepuppetmaster':
-    user        => 'root',
-    minute      => $puppetmaster_update_cron_interval[min],
-    hour        => $puppetmaster_update_cron_interval[hour],
-    monthday    => $puppetmaster_update_cron_interval[day],
-    month       => $puppetmaster_update_cron_interval[month],
-    weekday     => $puppetmaster_update_cron_interval[weekday],
-    command     => 'flock -n /var/run/puppet/puppet_run_all.lock bash /opt/system-config/production/run_all.sh >> /var/log/puppet_run_all_cron.log 2>&1',
-    environment => 'PATH=/var/lib/gems/1.8/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin',
+    ensure => absent,
   }
 
   logrotate::file { 'updatepuppetmaster':
@@ -111,19 +97,11 @@ class openstack_project::puppetmaster (
   }
 
   cron { 'deleteoldreports':
-    user        => 'root',
-    hour        => '3',
-    minute      => '0',
-    command     => 'sleep $((RANDOM\%600)) && find /var/lib/puppet/reports -name \'*.yaml\' -mtime +5 -execdir rm {} \;',
-    environment => 'PATH=/var/lib/gems/1.8/bin:/usr/bin:/bin:/usr/sbin:/sbin',
+    ensure => absent,
   }
 
   cron { 'deleteoldreports-json':
-    user        => 'root',
-    hour        => '3',
-    minute      => '0',
-    command     => 'sleep $((RANDOM\%600)) && find /var/lib/puppet/reports -name \'*.json\' -mtime +5 -execdir rm {} \;',
-    environment => 'PATH=/var/lib/gems/1.8/bin:/usr/bin:/bin:/usr/sbin:/sbin',
+    ensure => absent,
   }
 
   file { '/etc/puppet/hieradata':
@@ -346,11 +324,7 @@ class openstack_project::puppetmaster (
   }
 
   cron { 'expandgroups':
-    user        => 'root',
-    minute      => 0,
-    hour        => 4,
-    command     => '/usr/local/bin/expand-groups.sh >> /var/log/expand_groups.log 2>&1',
-    environment => 'PATH=/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin',
+    ensure => absent,
   }
 
   logrotate::file { 'expandgroups':
diff --git a/playbooks/roles/install-ansible/files/groups.yaml b/playbooks/roles/install-ansible/files/groups.yaml
index 4b1c4e6cf1..7f116ac4b1 100644
--- a/playbooks/roles/install-ansible/files/groups.yaml
+++ b/playbooks/roles/install-ansible/files/groups.yaml
@@ -6,7 +6,7 @@ groups:
   afsdb: inventory_hostname|regex_match('afsdb.*openstack.org')
   cacti: inventory_hostname|regex_match('cacti\d+\.openstack\.org')
   ci-backup: inventory_hostname|regex_match('backup\d+.*\ci\.openstack\.org')
-  disabled: inventory_hostname.startswith('backup') or inventory_hostname.startswith('wiki')
+  disabled: inventory_hostname.startswith('backup') or inventory_hostname.startswith('wiki') or inventory_hostname.startswith('puppetmaster')
   eavesdrop: inventory_hostname.startswith('eavesdrop')
   elasticsearch: inventory_hostname|regex_match('elasticsearch0[1-7]\.openstack\.org')
   ethercalc: inventory_hostname.startswith('ethercalc')