Skip to content
This repository has been archived by the owner on Jan 6, 2023. It is now read-only.

memory setting ignored for Vbox #115

Closed
dhoer opened this issue Jun 10, 2021 · 3 comments · Fixed by #118
Closed

memory setting ignored for Vbox #115

dhoer opened this issue Jun 10, 2021 · 3 comments · Fixed by #118

Comments

@dhoer
Copy link

dhoer commented Jun 10, 2021

Trying to set memory with the following

driver:
  name: vagrant
  provider:
    name: VBox
platforms:
  - name: ubuntu14
    box: ubuntu/trusty64
    memory: 8192

But the memory remains at 512.

@apatard
Copy link
Member

apatard commented Jun 10, 2021

As may be seen in examples of Vagrantfiles with virtualbox here: https://www.vagrantup.com/docs/providers/virtualbox/configuration, the provider name is virtualbox (all lower case), so your molecule.yml should look like:

driver:
  name: vagrant
  provider:
    name: virtualbox
platforms:
  - name: ubuntu14
    box: ubuntu/trusty64
    memory: 8192

For now, using the wrong provider name will still get a VM created, even if the settings are kind-of ignored. I've been wondering for quite some time to add to the vagrant up command the --provider $providername argument which will possibly result in an error if the provider name is wrong. Unfortunately it may break some people setup....

@dhoer
Copy link
Author

dhoer commented Jun 11, 2021

Thanks for the quick response. Having a warning about provider name mismatch might be a good start, or a deprecation warning. I don't remember where I found the configuration example that had VBox, but an example on the readme page would be a nice-to-have. If I have time this weekend, I will submit a PR to add it.

@konstruktoid
Copy link
Contributor

konstruktoid commented Jun 14, 2021

Related regarding the usage of VBox: #91, #91 (comment)

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 a pull request may close this issue.

3 participants