Upstart script puppet agent

The attachment "debdiff for puppet" of this bug report has been identified as being a patch in the form of a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-sponsors team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

Hi Glenn

Thanks for taking the time to prepare this update for the puppet packaging.

Couple of comments:

1) By default the init script for the puppet agent does not start the agent; this is set in /etc/default/puppet - the upstart configuration needs to exhibit the same behaviour.

2) The puppet package also has two other init scripts; we should switch the package over wholesale to upstart - no just one agent IMHO.

Although I would love to see the puppet packages using upstart I think its a bit late in the precise development cycle to accept this change.

I suggest that we land an appropriate change once Ubuntu Q development opens.

Unsubscribing ubuntu-sponsors for the time being.

Hi James,

Thanks for your feedback!

1) The upstart script has been extended with daemon checks, checks on the defaults file flags and also give error outputs. I added the puppet.conf upstart script as attachment.
2) Since mongrel support will be deleted from debian (http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=663921) im assuming that ubuntu 12.10 wont have mongrel support either. This would mean that the puppetmaster upstart script wont need to support the mongrel backend and only standard webrick. See attachment puppetmaster.conf
3) Also added the puppetqd upstart script as attachment in puppetqd.conf.
4) It would be nice to have an upstart script for puppetmaster-passenger too. But i think this would be the apache2 upstart script. Is there any eta on the apache2 upstart script, or if someone is working on that already?

Status changed to 'Confirmed' because the bug affects multiple users.