summaryrefslogtreecommitdiff
path: root/manifests/debian.pp
diff options
context:
space:
mode:
authorMicah Anderson <micah@riseup.net>2012-08-09 16:03:01 -0400
committervarac <varacanero@zeromail.org>2012-08-10 18:10:30 +0200
commitc5a275505bf2aafb9378d923985f9915e81ce7a7 (patch)
treeeca1c1102adab67a3b6cb387d236d8355116738d /manifests/debian.pp
parentdffec4d8134be8a2eb2bb3a8063b2d60d67308f1 (diff)
fix puppetmaster service definition for passenger
Because passenger mode doesn't have its own daemon, it was a hack before to have the service definition for 'puppetmaster' be actually looking in the process list for 'apache2'. This sort of worked, but not if you need to notify the service for a restart. It also didn't actually work, because the hasstatus parameter was set to true, which meant that on every run, puppet did a /etc/init.d/puppetmaster status and found that it was not running and then tried to start it by doing /etc/init.d/puppetmaster start. That doesn't work because its turned off in /etc/default/puppetmaster when puppetmaster_mode='passenger'. So... this commit removes that hacky service definition and instead just requires the apache::base class, providing the apache service monitoring that is needed when you are running puppetmaster_mode='passenger'. It also has to pull up the Service[puppet] override which was adding the puppetmaster service, which makes no sense because there is no service. Finally, in order to notify it for changes, we need to use a selector to determine how to reload things based on puppetmaster_mode. Conflicts: manifests/puppetmaster/linux.pp
Diffstat (limited to 'manifests/debian.pp')
0 files changed, 0 insertions, 0 deletions