search

UtahDave / salt-vagrant-demo

Demo of Salt in Vagrant. 1 master and 2 minions
Apache License 2.0
247 stars 191 forks source link

changes install type from "stable" to "git" to workaround salt issue #25042 #25

Closed jacobhammons closed 8 years ago

jacobhammons commented 8 years ago

https://github.com/saltstack/salt/issues/25042#issuecomment-222512976

@UtahDave Some users are now having issues using this with the get started guide (see issue above), which I was able to confirm. I tried upgrading the box, upgrading virtualbox and vagrant, and nothing did the trick.

After testing, changing the vagrantfile to install from git fixed the issue, I tested these changes with the latest vagrant/virtualbox and it worked great. The minions respond immediately.

Do you mind merging this until the referenced issue is fixed?

I also removed the -P flag from the bootstrap options in this PR since it is no longer needed.

UtahDave commented 8 years ago

I added more ram to the VM and that fixed the problem. I'm going to close this PR as I'd rather have a package install than a git install, by default.

jacobhammons commented 8 years ago

Awesome! Thanks for sorting this out.