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

Windows: resolved merge conflicts and added tests #196

Merged
merged 7 commits into from Jul 23, 2015

Conversation

gdavison
Copy link
Contributor

This adds support for installing on Windows using the install_method of windows. It uses the Chocolatey package manager.

I have made the Windows tests conditional on setting the TEST_WINDOWS environment variable to a truthy value. I tried using vagrant boxes from [https://github.com/joefitzgerald/packer-windows] and [https://github.com/boxcutter/windows], but I was not able to make them work. The box used is approximately 3 GB to download. Provisioning only seems to work if you login at boot time; annoying, but it works.

@gdavison
Copy link
Contributor Author

The box I was using appears to have been taken offline today. I've changed to a new box from the same source. You no longer need to log in, but kitchen seems to try on first run to use ssh to connect. I cancel the first run with ^C, then re-run the test, and it uses winrm. No idea why.

@gdavison gdavison changed the title resolved merge conflicts and added tests Windows: resolved merge conflicts and added tests Jul 17, 2015
@gdavison gdavison mentioned this pull request Jul 17, 2015
@johnbellone johnbellone added this to the 0.10.2 milestone Jul 21, 2015
@johnbellone
Copy link
Contributor

What's the best way to test this?

@gdavison
Copy link
Contributor Author

I've fixed some of the Windows VM stuff by manually setting some values that I thought Kitchen was supposed to set. I'm verifying that all tests pass right now, and will re-push when that's done.

@gdavison
Copy link
Contributor Author

It's all updated. You can run TEST_WINDOWS=1 kitchen test "^[^rs].*(ub|win)". It skips the centos, runit, and source tests, and runs the windows test.

The Windows box can take quite a while to download. 20-30 minutes at home, but it was several hours before I had to cancel it when leaving the office.

@johnbellone
Copy link
Contributor

This is failing for me. Are you using ChefDK?

@gdavison
Copy link
Contributor Author

Yes, ChefDK 0.6.2 on Mac OS 10.10. What errors are you getting?

@@ -10,9 +10,22 @@ platforms:
- name: ubuntu-12.04
- name: centos-7.0
- name: centos-6.5
<% if ENV['TEST_WINDOWS'] %>

This comment was marked as outdated.

johnbellone added a commit that referenced this pull request Jul 23, 2015
Windows: resolved merge conflicts and added tests
@johnbellone johnbellone merged commit 97366d1 into sous-chefs:master Jul 23, 2015
@lock
Copy link

lock bot commented Apr 25, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Apr 25, 2020
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 this pull request may close these issues.

None yet

2 participants