From 346e0e0c5819107474b47025ac7fadaa751f63a5 Mon Sep 17 00:00:00 2001 From: Micah Anderson Date: Wed, 22 Aug 2012 14:12:34 -0400 Subject: switch to $::operatingsystem as suggested by ng in https://labs.riseup.net/code/issues/4029 --- manifests/puppetmaster/passenger.pp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'manifests') diff --git a/manifests/puppetmaster/passenger.pp b/manifests/puppetmaster/passenger.pp index 9f4d1e8..e026a62 100644 --- a/manifests/puppetmaster/passenger.pp +++ b/manifests/puppetmaster/passenger.pp @@ -7,7 +7,7 @@ class puppet::puppetmaster::passenger inherits puppet::puppetmaster::base { # A reference configuration is available at : # http://github.com/reductivelabs/puppet/tree/master/ext/rack - case $operatingsystem { + case $::operatingsystem { debian: { package { 'puppetmaster-passenger': ensure => installed } file { -- cgit v1.2.3 From 0097eeef0b24a85d6f067bbf5b126d3fa22f1c15 Mon Sep 17 00:00:00 2001 From: Micah Anderson Date: Wed, 22 Aug 2012 14:40:59 -0400 Subject: implement fixes from #3514 --- manifests/debian.pp | 18 ++++++++---------- manifests/puppetmaster/debian.pp | 27 ++++++++++++++++----------- 2 files changed, 24 insertions(+), 21 deletions(-) (limited to 'manifests') diff --git a/manifests/debian.pp b/manifests/debian.pp index f0479ae..e24a1bc 100644 --- a/manifests/debian.pp +++ b/manifests/debian.pp @@ -1,21 +1,19 @@ class puppet::debian inherits puppet::linux { file { '/etc/default/puppet': - source => [ "puppet:///modules/site-puppet/client/debian/${fqdn}/puppet", - "puppet:///modules/site-puppet/client/debian/${domain}/puppet", - "puppet:///modules/site-puppet/client/debian/puppet", + source => [ "puppet:///modules/site_puppet/client/debian/${fqdn}/puppet", + "puppet:///modules/site_puppet/client/debian/${domain}/puppet", + "puppet:///modules/site_puppet/client/debian/puppet", "puppet:///modules/puppet/client/debian/puppet" ], notify => Service[puppet], owner => root, group => 0, mode => 0644; } - case $lsbdistcodename { - squeeze,sid: { - $real_puppet_hasstatus = true - } - default: { - $real_puppet_hasstatus = false - } + if versioncmp($puppetversion,'2.6') >= 0 { + $real_puppet_hasstatus = true + } + else { + $real_puppet_hasstatus = false } Service[puppet]{ diff --git a/manifests/puppetmaster/debian.pp b/manifests/puppetmaster/debian.pp index 853a0e6..fc24221 100644 --- a/manifests/puppetmaster/debian.pp +++ b/manifests/puppetmaster/debian.pp @@ -1,21 +1,26 @@ -class puppet::puppetmaster::debian inherits puppet::puppetmaster::package { +class puppet::puppetmaster::debian { + include puppet::puppetmaster::package + include puppet::puppetmaster::linux + if $puppetmaster_mode != 'passenger' { - case $lsbdistcodename { - squeeze,sid: { - Service['puppetmaster'] { hasstatus => true, hasrestart => true } - } + if $puppet_majorversion >= '2.6' { + Service['puppetmaster'] { hasstatus => true, hasrestart => true } } } + if $puppetmaster_mode == 'passenger' { + $puppetmaster_default_nofity = 'Exec[notify_passenger_puppetmaster]' + } + file { '/etc/default/puppetmaster': - source => [ "puppet:///modules/site-puppet/master/debian/${fqdn}/puppetmaster", - "puppet:///modules/site-puppet/master/debian/${domain}/puppetmaster", - "puppet:///modules/site-puppet/master/debian/puppetmaster", + source => [ "puppet:///modules/site_puppet/master/debian/${fqdn}/puppetmaster", + "puppet:///modules/site_puppet/master/debian/${domain}/puppetmaster", + "puppet:///modules/site_puppet/master/debian/puppetmaster", "puppet:///modules/puppet/master/debian/puppetmaster" ], - notify => $puppetmaster_mode ? { - 'passenger' => Exec['notify_passenger_puppetmaster'], - default => Service[puppetmaster], + notify => $puppetmaster_default_nofity ? { + '' => Service[puppetmaster], + default => Exec['notify_passenger_puppetmaster'] }, owner => root, group => 0, mode => 0644; } -- cgit v1.2.3