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

Lower-case provider name (vSphere=>vsphere) #87

Merged
merged 1 commit into from
Sep 3, 2014
Merged

Lower-case provider name (vSphere=>vsphere) #87

merged 1 commit into from
Sep 3, 2014

Conversation

mkuzmin
Copy link
Contributor

@mkuzmin mkuzmin commented Aug 28, 2014

As discussed in #86, when a box is not installed locally, and Vagrant downloads it from remote server, it checks the provider name.
If metadata.json within a box file contains incorrect value, it causes errors like

(manage-vsphere=)$ vagrant up dev --provider vsphere
Bringing machine 'dev' up with 'vsphere' provider...
==> dev: Running triggers before up...
==> dev: Box 'vsphere' could not be found. Attempting to find and install...
    dev: Box Provider: vsphere
    dev: Box Version: >= 0
==> dev: Adding box 'vsphere' (v0) for provider: vsphere
    dev: Downloading: file:///Users/savoie/Dropbox/projects/vagrant-nsidc/vsphere/vsphere.box
The box you attempted to add doesn't match the provider you specified.

Provider expected: vsphere
Provider of box: vSphere

The check is case-sensitive, so the value in metadata must be equal to the name declared by a pugin at https://github.com/nsidc/vagrant-vsphere/blob/master/lib/vSphere/plugin.rb#L16

  Lower-case is required by Vagrant when a box is not installed locally, and it's downloaded from a remote URL.
@flamingbear
Copy link
Member

Ok. it looks like that's baked into the box I've been given I'll have to work with our admins to get one with the correct metadata.

I just got your note on #86, so sounds like we're on the same page now. Thanks.

flamingbear added a commit that referenced this pull request Sep 3, 2014
Lower-case provider name (vSphere=>vsphere)
@flamingbear flamingbear merged commit 0aac41d into nsidc:master Sep 3, 2014
@mkuzmin mkuzmin deleted the provider-name branch September 9, 2014 20:40
@mkuzmin mkuzmin deleted the provider-name branch September 9, 2014 20:40
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.

2 participants