Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Vagrant Development Envoirment - Ansible Setup with Windows Host #88

Merged
merged 4 commits into from
Feb 10, 2014

Conversation

crankeye
Copy link
Contributor

@crankeye crankeye commented Feb 9, 2014

I've added a script to the Vagrantfile for provisioning with Ansible when the host OS is Windows. Ansible currently does not support Windows as the control machine. The work around involves installing Ansible on the guest machine and running the provisioning from the guest machine.

I don't particularly like the if/else logic in the Vagrant file but AFIAK this is the best solution for this particular situation.

Adding notes about windows only commands when installing the node.js packages and a brief blurb about the Ansible install on windows.
…a Windows host. Ansible does not currently support Windows as the control machine.
@megamattron
Copy link
Member

Thanks for the PR! I'm going to have to plead ignorance on the details of vagrant, but there appears to be a few PRs kicking around related to it so maybe you guys could collaborate to figure out what to do? I'm happy to merge this if you guys say it's good to go as well, would be good to get @crankeye on the approved users for his efforts.

cc'ing other people who have contributed vagrant related stuff: @shedd @diminoten

@shedd
Copy link
Contributor

shedd commented Feb 9, 2014

Thanks, @crankeye - I wasn't aware of the lack of Windows support in Ansible.

The strategy that you took seems to match the other examples that I found for provisioning a VM guest using Ansible while on a Windows host (I looked at https://github.com/KSid/windows-vagrant-ansible and https://github.com/Taytay/vagrant-ansible-postgres).

Makes sense to me. Thanks for making the Vagrant dev environment more cross-platform friendly!

+1

megamattron pushed a commit that referenced this pull request Feb 10, 2014
@megamattron megamattron merged commit 60b13c8 into larvalabs:master Feb 10, 2014
@megamattron
Copy link
Member

Thanks for the input @shedd - I've merged and you should have access now @crankeye

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants