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

private_network is specific to Virtualbox #519

Closed

Conversation

DavidSchargel
Copy link

Move private_network config into virtualbox block since it is not needed for VMware. Minor, but eliminates NFS warning messages on VMware 'vagrant up'.

@DavidSchargel
Copy link
Author

My first PR ever…hope I did this right.

@DavidSchargel
Copy link
Author

As per https://www.vagrantup.com/docs/synced-folders/nfs.html, private_network is a prerequisite only due to a limitation of VirtualBox's built-in networking, not for VMware.

@austinpray
Copy link
Contributor

Doesn't the fixed IP have to be there for hostsupdater to work?

@DavidSchargel
Copy link
Author

Hmmm… @austinpray…I'm not even sure if that hostsupdater: 'skip' works any longer with the switch from vagrant-hostsupdater to vagrant-hostmanager a few months ago in #442. Since I'm not on VirtualBox, I can't test to see what happens definitively, but I would suspect that hostsupdater: 'skip' isn't working any longer anyway.

However, for similar hostmanager functionality, I wonder if Virtualbox users should see the addition of config.hostmanager.ignore_private_ip = false to the config.hostmanager block of the VagrantFile as per the vagrant-hostmanager docs. I would think that VirtualBox users would have reported something earlier if this was the case.

@QWp6t
Copy link
Member

QWp6t commented Mar 18, 2016

hostsupdater: 'skip' was added because we switched to hostmanager. #458

@DavidSchargel
Copy link
Author

Got it; I no longer have vagrant-hostsupdater installed. hostsupdater: 'skip' obviously needs to stay in place then.

Since 1) that hostsupdater: "skip" line is only available when used on the config.vm.network :private_network entry I am moving, and 2) my PR is VMware-specific and only suppresses a warning message, let's not make that change. How do I withdraw/close this PR?

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