You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Out of the box, the salt plugin doesn't appear to properly seed minion keys on the master when the seed_master option is used (the root cause of this issue).
After some digging, I have determined that the seed_master section of Vagrant\embedded\gems\gems\vagrant-1.5.2\plugins\provisioners\salt\provisioner.rb is the culprit:
For example, changing seed_dir = "/tmp/minion-seed-keys" to seed_dir = "/etc/salt/pki/master/minions" eliminates the issue.
Alternatively, the salty-vagrant version (which still has another bug) first uploads the files to a tmp directory and then moves them to the correct final location.
Obviously, I'm not picky which route is used to solve the problem, but newbies to Salt and Vagrant together will rejoice at avoiding a bug in a basic feature that is often used in examples.
The text was updated successfully, but these errors were encountered:
claytondaley
changed the title
Salty Plugin: seed_master uses wrong directory
Salt Plugin: seed_master uses wrong directory
Apr 17, 2014
Rookie mistake. I finally figured out that the -k option should be copying files from the temp directory into the final directory and confirmed that the bootstrap script is getting the right values... and then trying to move an incorrect filename. Will follow up there.
Out of the box, the salt plugin doesn't appear to properly seed minion keys on the master when the seed_master option is used (the root cause of this issue).
After some digging, I have determined that the seed_master section of
Vagrant\embedded\gems\gems\vagrant-1.5.2\plugins\provisioners\salt\provisioner.rb
is the culprit:seed_dir = "/tmp/minion-seed-keys"
toseed_dir = "/etc/salt/pki/master/minions"
eliminates the issue.Obviously, I'm not picky which route is used to solve the problem, but newbies to Salt and Vagrant together will rejoice at avoiding a bug in a basic feature that is often used in examples.
The text was updated successfully, but these errors were encountered: