Merge "Move puppet service disablement to o_p::server"

install_puppet.sh

@@ -226,6 +226,11 @@ EOF
         --assume-yes install -y --force-yes puppet git $rubypkg
     # Wipe out templatedir so we don't get warnings about it
     sed -i '/templatedir/d' /etc/puppet/puppet.conf
+    if [ -f /bin/systemctl ]; then
+        systemctl disable puppet
+    else
+        service puppet disable
+    fi
 }
 
 function setup_puppet_opensuse {

modules/openstack_project/manifests/server.pp

@@ -161,6 +161,25 @@ class openstack_project::server (
     require  => Class['pip'],
   }
 
+  ###########################################################
+  # Turn off puppet service
+
+  service { 'puppet':
+    ensure => stopped,
+    enable => false,
+  }
+
+  if $::osfamily == 'Debian' {
+    file { '/etc/default/puppet':
+      ensure  => present,
+      owner   => 'root',
+      group   => 'root',
+      mode    => '0444',
+      source  => 'puppet:///modules/openstack_project/puppet.default',
+      replace => true,
+    }
+  }
+
   class { 'openstack_project::template':
     iptables_public_tcp_ports => $iptables_public_tcp_ports,
     iptables_public_udp_ports => $iptables_public_udp_ports,

modules/openstack_project/manifests/template.pp

@@ -153,16 +153,6 @@ class openstack_project::template (
       source  => 'puppet:///modules/openstack_project/90no-translations',
       replace => true,
     }
-
-    file { '/etc/default/puppet':
-      ensure  => present,
-      owner   => 'root',
-      group   => 'root',
-      mode    => '0444',
-      source  => 'puppet:///modules/openstack_project/puppet.default',
-      replace => true,
-    }
-
   }
 
   if ($::operatingsystem == 'CentOS') {
@@ -175,10 +165,6 @@ class openstack_project::template (
       replace => true,
     }
   }
-  service { 'puppet':
-    ensure => stopped,
-    enable => false,
-  }
 
   ###########################################################