search

UtahDave / salt-vagrant-demo

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

Added Salt Configuration #24

Closed maradwan closed 8 years ago

maradwan commented 8 years ago

Dear Team, I have used option 2. One Vagrant for Salt Master and another one for Salt minion. You need to do the following : 1- #vagrant up . It takes time to install the packages . 2- Please use http://192.168.50.11/

Thanks Mohamed

UtahDave commented 8 years ago

Hi @maradwan

I'm hesitant to merge this pull request. I'd like to keep this demo simple and not install mysql and a webserver, etc. Can you explain why we should add all these packages?

maradwan commented 8 years ago

Hi David,

I was working on the app. The app is needs mysql and webserver.

Thanks Mohamed

On Tue, Jun 7, 2016 at 10:33 PM, David Boucha notifications@github.com wrote:

Hi @maradwan https://github.com/maradwan

I'm hesitant to merge this pull request. I'd like to keep this demo simple and not install mysql and a webserver, etc. Can you explain why we should add all these packages?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/UtahDave/salt-vagrant-demo/pull/24#issuecomment-224405319, or mute the thread https://github.com/notifications/unsubscribe/AIyf6fC5rr0Xxf2lNbgRIoOBYpxdXYxCks5qJdWzgaJpZM4HasjJ .

UtahDave commented 8 years ago

Hi, @maradwan, thanks so much for your PR. I think it's quite well done. I think the idea of having just one minion is a pretty good one, which I will definitely consider. One of the goals of this demo is for it to be simple and clear. I think I'm going to close this PR rather than merge it because of the mysql and webserver additions. I think that would be more appropriate for another demo, which i'd be willing to collaborate on if you're interested.

Let me know what you think.