-
Notifications
You must be signed in to change notification settings - Fork 148
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
Merged configuration from box vagrantfiles not used #182
Comments
Opened that in #131 |
Hey @marek-obuchowicz / @dkarlovi, Can you test amontalban@09ef519 on your setup? I had same issue (Configuring vagrant-hostmanager in base box) and that fixed. I'm not a developer and my Ruby skills are super limited but I hope it helps. If that works I can open an PR and maybe @seth-reeser can merge it. Thanks! |
@amontalban I will try it out monday and let you know, thanks. |
@amontalban I'm sorry about this taking quite so long, but I do confirm that it does in fact work, I've removed
from my (thin) project
Please, create a PR so we can get this fixed. Thank you, great job! |
@dkarlovi Sorry for the long delay, sure let me push a PR. Thanks! |
Sorry, it works it seems that Vagrant have an issue when installing plugins from remote repositories. Check hashicorp/vagrant#8210. For now you can download this patched plugin and install it with: You can download it from: |
I have configured hostmanager in Vagrantfile, then packaged it into box. When I'm creating a vm using
vagrant init <boxname>; vagrant up
- it doesn't respect hostmanager configuration from box file.I guess it comes from https://github.com/devopsgroup-io/vagrant-hostmanager/blob/master/lib/vagrant-hostmanager/action/update_host.rb#L13 -> it calls env[:global_env].vagrantfile which, according to documentation, "Represents the default Vagrantfile, or the Vagrantfile that is in the working directory or a parent of the working directory of this environment." - so it doesn't consider the value from merged configurations of all Vagrantfiles, just the one from file in current directory.
The text was updated successfully, but these errors were encountered: