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

Vagrantfile added? #1

Open
alvagante opened this issue Nov 18, 2015 · 3 comments
Open

Vagrantfile added? #1

alvagante opened this issue Nov 18, 2015 · 3 comments

Comments

@alvagante
Copy link

Generally in the control-repos I do for different Puppet setups, I include a Vagrantfile and relevant scripts to be able to test locally the same Puppet and data. Would a PR with such an addition be welcomed? If so I can provide one.

@bkc1
Copy link

bkc1 commented Dec 11, 2015

+1 for a Vagrantfile to be included

@npwalker
Copy link
Contributor

@alvagante I think we're trying to prevent this control-repo from doing too much. How tightly coupled does the Vagrantfile need to be to the control-repo?

Could there be a seperate repo for the Vagrantfile that includes instructions for how to add it to the control-repo?

Or could it be expected that the control-repo was cloned into a directory within the Vagrant directory?

I understand these might not be as elegant of a solution but going from a single clone to clone vagrantfile repo, cd, clone control-repo doesn't seem that bad. What do you think?

@alvagante
Copy link
Author

@npwalker Well, in all my "control-repos" for different projects (didn't call them this way but they were doing the same thing) I've always found very useful a local vagrant environment where I could test quickly my changes to puppet code and data before committing them.
Note that in Vagrant you'd have a multi VM environment with one vm per role, so the vagrant part is an essential part of the repo and has some coupling with the used roles.
Having a separated Vagrant repo would do the same, but it would need some sync between the two repos.
my2c

chrismatteson pushed a commit to chrismatteson/control-repo that referenced this issue Jun 3, 2016
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

No branches or pull requests

3 participants