add Salt support to cloud-init

Bug #927795 reported by Jeff Bauer on 2012-02-06
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
cloud-init
Undecided
Unassigned

Bug Description

Salt is a zeromq-based configuration management tool written in Python.

http://saltstack.org/

It would be useful to have Salt minions supported in cloud-init, similar to Puppet and Chef.

Related branches

Jeff Bauer (jbauer) wrote :

I've pushed a salt branch to launchpad:

    https://code.launchpad.net/~jbauer/cloud-init/salt

Currently only support for minions, as that would be the normal use case for cloud instances.

Scott Moser (smoser) wrote :

Jeff,
 I took a look, and generally looks ok.
 2 things:
   a.) I'm guessing we should then add 'salt' to the cloud_config_modules list
   b.) I haven't checked the packages of salt, but generally installation of a package in debian is supposed to start the service. Thus, the 'check_call' to 'service start-minion start', would fail in an upstart world, where 'start' on an already started service will exit error.
  Ie:
     $ sudo service ssh start; echo $?
     start: Job is already running: ssh
    1

So, I'd like for that to either check if running and start if not, or somehow handle that case, so that if the packages did automatically start that daemon, you wouldn't cause failure of this cloud-config module even though everything was actually good.

Scott Moser (smoser) wrote :

fix committed (sans change for 'service') in upsream 513.

Changed in cloud-init:
status: New → Fix Committed
Scott Moser (smoser) on 2012-04-11
Changed in cloud-init:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers