Skip to content
This repository has been archived by the owner on Feb 18, 2021. It is now read-only.

Adding additional providers #125

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

Adding additional providers #125

wants to merge 1 commit into from

Conversation

beardedeagle
Copy link
Contributor

  • Added support for Parallels
  • Added support for VMWare
  • Added support for Parallels specific image
  • Changed VirtualBox specifics to easer to read and use helpers

- Added support for Parallels
- Added support for VMWare
- Added support for Parallels specific image
- Changed VirtualBox specifics to easer to read and use helpers
@Lothiraldan
Copy link
Owner

Thank you very much for the contribution! I don't have experience with other provider than virtualbox when using vagrant, how are we supposed to use parallels instead of virtualbox? Comment the virtualbox part and uncomment the paralles part?

@beardedeagle
Copy link
Contributor Author

You can uncomment all of them all at once if you would like as calling with a specific provider will ignore the other provider directives, I just commented them out for this commit. You basically have to preconfigure your provider or call it during your vagrant up command like such:
vagrant up --provider=parallels

I have a work provided mac that comes with parallels so I usually use it over VirtualBox or VMWare since the integration into the mac desktop is superb. I have tested these configurations and they work. I will link the docs below if you would like to look though:

http://parallels.github.io/vagrant-parallels/docs/usage.html
https://docs.vagrantup.com/v2/vmware/usage.html

@Lothiraldan
Copy link
Owner

Will it works if the providers specific sections are commented?

@beardedeagle
Copy link
Contributor Author

They would need to uncomment if they were using a provider other than VirtualBox before running the vagrant up command.

@beardedeagle
Copy link
Contributor Author

Do you need me to do anything else to get this merged?

@Lothiraldan
Copy link
Owner

Hi @beardedeagle, I wanted to try to play with multiple providers before merging, will do that right now.

@Lothiraldan
Copy link
Owner

I give it a try, I would prefer to have all providers uncommented as it isn't disturbing when we don't have parallels installed for example.

For the parallels custom vm box, we may use the override argument like that:

  # Parallels provider specifics
  config.vm.provider "parallels" do |prl, override|
   prl.memory = 1024
   prl.cpus = 2
   # Parallels specific Ubuntu 14.04 Vagrant box
   override.vm.box = "parallels/ubuntu-14.04"
  end

to use the ubuntu/trusty64 in all case except with parallels. With this modification I see no barrier to uncomment all providers blocks, maybe we just need to put a phrase in the README saying to users can use the --provider argument of vagrant to choose the one they want.

Could you give it a try with the override modification?

@beardedeagle
Copy link
Contributor Author

yeah, I will get this done and updated asap.

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

Successfully merging this pull request may close these issues.

2 participants