cloud-init

add Salt support to cloud-init

Bug #927795 reported by Jeff Bauer on 2012-02-06

This bug affects 2 people

Affects		Status	Importance	Assigned to	Milestone
	cloud-init	Fix Released	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.

Tags:

Related branches

lp:ubuntu/precise/cloud-init

Revision history for this message

Jeff Bauer (jbauer) wrote on 2012-02-11:

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.

Revision history for this message

Scott Moser (smoser) wrote on 2012-02-15:

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.

Revision history for this message

Scott Moser (smoser) wrote on 2012-02-15:

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