search

chef / omnibus-ctl

Provides service control for omnibus packages
Apache License 2.0
23 stars 24 forks source link

Allow omnibus-ctl to be invoked without having the service management commands. #5

Closed jamesc closed 10 years ago

jamesc commented 10 years ago

Addons (e.g. pushy/reporting/manage) don't manage their own services but use the private-chef-ctl runit.

This patch allows for a second initialization option 'service_commands' (true by default to mimic current behaviour) where the ctl command does not have any runit related commands. Includes updated tests

Note: This doesn't include a new wrapper script ala. omnibus-ctl - I'm dropping off a script setting the options directly in the relevant omnibus software config (e.g. omnibus-reporting-ctl)

cc @christophermaier, @schisamo

christophermaier commented 10 years ago

Looks good. Haven't run it, but this is in line with what @jamesc and I have talked about in the past.

schisamo commented 10 years ago

This is great, I was just thinking about the same thing last week!