-
Notifications
You must be signed in to change notification settings - Fork 60
51update #25
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
base: master
Are you sure you want to change the base?
51update #25
Conversation
capturing links to the 5.1.16 while thinking about it.
|
just tested the branch. works :-) |
not tested yet in a good way
5.1.16 update turned into a 5.1.18 update that includes a working windows 2016 kitchen test install. The windows verification still does not work. I don't know trick to writing a good package verification that works for linux and windows.
|
@chris-orchard I can merge this PR. I do not have a lot of time to test this though. Can someone else test this branch? |
|
@bradleyd what's the involved in testing it? Is this something I could do easily? @ehaselwanter you tested this previously but there have since been further commits by @dayne. Are you able to test it again so we can get it merged please? |
|
@bradleyd @ehaselwanter Can either of you help please? |
Thanks for your Virtualbox Cookbook.
This pull request updates the cookbook to install the latest VirtualBox (5.1.14) and updates test kitchen systems.
Updated the test kitchen do ubuntu 16.04, debian 8.6, centos 6.8 and centos 7.2. All of these converge successfully.
Converted to inspec tests to verify install of 5.1 happened.
With the attached updates I've found it to be working great for my Ubuntu 16.10 system.
Did not try to review how well (or not) this works for Windows or OSX.
Let me know if you'd like any more details or tweaks.