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

minishift config view is out of order #699

Closed
burrsutter opened this issue Apr 6, 2017 · 3 comments
Closed

minishift config view is out of order #699

burrsutter opened this issue Apr 6, 2017 · 3 comments

Comments

@burrsutter
Copy link

burrsutter commented Apr 6, 2017

$ ./minishift version
You need to run 'minishift setup-cdk' first to install required CDK components.

$ ./minishift setup-cdk
Setting up CDK 3 on host using '/Users/burr/.minishift' as Minishift's home directory
Copying minishift-rhel7.iso to '/Users/burr/.minishift/cache/iso/minishift-rhel7.iso'
Copying oc to '/Users/burr/.minishift/cache/oc/v3.4.1.2/oc'
Creating configuration file '/Users/burr/.minishift/config/config.json'
Creating marker file '/Users/burr/.minishift/cdk'
CDK 3 setup complete.

$ ./minishift config view
- iso-url: file:///Users/burr/.minishift/cache/iso/minishift-rhel7.iso
- vm-driver: xhyve

$ ./minishift config set memory 6000
To apply the changes, you must delete the current VM with `minishift delete` and start a new VM with `minishift start`.

$ ./minishift config view
- vm-driver: xhyve
- iso-url: file:///Users/burr/.minishift/cache/iso/minishift-rhel7.iso
- memory: 6000

$ ./minishift config set cpus 2
To apply the changes, you must delete the current VM with `minishift delete` and start a new VM with `minishift start`.

$ ./minishift config view
- cpus: 2
- iso-url: file:///Users/burr/.minishift/cache/iso/minishift-rhel7.iso
- memory: 6000
- vm-driver: xhyve

$ ./minishift config set 
usage: minishift config set PROPERTY_NAME PROPERTY_VALUE

$ ./minishift config set vm-driver virtualbox
To apply the changes, you must delete the current VM with `minishift delete` and start a new VM with `minishift start`.
@jorgemoralespou
Copy link
Contributor

What are your expectations here? Why is providing ordered config valuable?
You need to provide more details to better qualify the fix and plan accordingly.

@LalatenduMohanty
Copy link
Member

@burrsutter I have same questions as @jorgemoralespou mentioned. Please provide more information.

@hferentschik
Copy link
Member

I think the expectations is that the order of the properties is always the same. It is random and depend on the iteration order of the map in which the properties are read. This has been resolved in rc1.

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

No branches or pull requests

4 participants