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

Enable ansible access during provisioning #88

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jean
Copy link

@jean jean commented Jun 14, 2017

Without this, Ansible can't connect to the created VM.
This is a fix to #86.
A better fix is probably #24 which modifies the
box creation, rather than adding provisioning after the fact, but that
has gone unmerged since 2015.

Without this, Ansible can't connect to the created VM.
This is a fix to mwrock#86
A better fix is probably
mwrock#24 which modifies the
box creation, rather than adding provisioning after the fact, but that
has gone unmerged since 2015.
@mwrock
Copy link
Owner

mwrock commented Jun 14, 2017

Thanks for taking the time to put this together. My goal with these templates is that they be as generic as possible so I'd prefer not to include specific IP addresses or scripts specific to any individual configuration management tool. I'm certainly open to enabling winrm over https but I'd prefer to see that in the packer setup and not the Vagrantfile.

@jean
Copy link
Author

jean commented Jun 15, 2017

@mwrock I agree that this belongs in the packer setup, I think that's what #24 is meant to do?
In the meantime, users end up with a box that's unreachable by ansible unless they discover the required changes to the Vagrantfile, so I submit this PR as a way to get them underway more easily.

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.

None yet

2 participants