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

Improve windows support of omnibus installer #890

Merged
merged 2 commits into from
Aug 10, 2016
Merged

Conversation

ksubrama
Copy link

@ksubrama ksubrama commented Aug 8, 2016

No description provided.

@ksubrama ksubrama added the Type: Enhancement Improves an existing feature label Aug 8, 2016
@ksubrama
Copy link
Author

ksubrama commented Aug 8, 2016

@stevendanna @chris-rock @arlimus @alexpop

I still need to fix the icons but the rest of it should be ready to look at.


suites:
- name: default
- name: inspec
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

any reason why we renamed that?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

None in particular - just cosmetic. If I've a bunch of VMs running, setting the suite name correctly helps identify the VM inside virtual box. Otherwise everything looks like "kitchen-omnibus-default-windows-2012r2-standard..."

@stevendanna
Copy link
Contributor

Nice work!

@ksubrama ksubrama changed the title WIP: Improve windows support of omnibus installer - fix #889 Improve windows support of omnibus installer - fix #889 Aug 9, 2016
@chris-rock chris-rock changed the title Improve windows support of omnibus installer - fix #889 Improve windows support of omnibus installer Aug 10, 2016
@chris-rock
Copy link
Contributor

fixes #889

@chris-rock chris-rock merged commit d4850a0 into master Aug 10, 2016
@chris-rock chris-rock deleted the ksubrama/win_omnibus branch August 10, 2016 20:38
@chris-rock chris-rock modified the milestone: 0.30.0 Aug 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Enhancement Improves an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants