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

Initial windows support #1053

Merged
merged 17 commits into from
Jun 4, 2018
Merged

Initial windows support #1053

merged 17 commits into from
Jun 4, 2018

Conversation

tas50
Copy link
Contributor

@tas50 tas50 commented Jun 1, 2018

This PR builds on the work done in Matt Wrock's https://github.com/mwrock/packer-templates. It's been modified to use Chef for almost all the work (minus final cleanup) and most of the things we did in boxcutter-windows have been moved over to the Chef recipe. Using Chef here is really nice not only because the complexity of the powershell/vbscript/batch melts away, but because we can easily test the logic in test kitchen without having to fully spin up a windows install. The feedback loop is pretty great, and after spending days waiting for boxcutter-windows installs to fail I'd never go back.

Signed-off-by: Tim Smith tsmith@chef.io

tas50 added 14 commits June 1, 2018 12:43
This has been hacked up a bit from Matt's initial work. I'm going to continue to hack it up to meet our needs

Signed-off-by: Tim Smith <tsmith@chef.io>
Signed-off-by: Tim Smith <tsmith@chef.io>
Signed-off-by: Tim Smith <tsmith@chef.io>
Signed-off-by: Tim Smith <tsmith@chef.io>
Signed-off-by: Tim Smith <tsmith@chef.io>
Signed-off-by: Tim Smith <tsmith@chef.io>
Signed-off-by: Tim Smith <tsmith@chef.io>
Signed-off-by: Tim Smith <tsmith@chef.io>
Signed-off-by: Tim Smith <tsmith@chef.io>
Signed-off-by: Tim Smith <tsmith@chef.io>
Signed-off-by: Tim Smith <tsmith@chef.io>
Signed-off-by: Tim Smith <tsmith@chef.io>
Signed-off-by: Tim Smith <tsmith@chef.io>
Signed-off-by: Tim Smith <tsmith@chef.io>
@tas50 tas50 changed the title WIP: Add windows support Initial windows support Jun 4, 2018
tas50 added 3 commits June 4, 2018 11:20
Signed-off-by: Tim Smith <tsmith@chef.io>
Signed-off-by: Tim Smith <tsmith@chef.io>
Signed-off-by: Tim Smith <tsmith@chef.io>
Copy link
Contributor

@cheeseplus cheeseplus left a comment

Choose a reason for hiding this comment

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

Long overdue and great work brining this all together @tas50

@cheeseplus cheeseplus merged commit c21e1a3 into master Jun 4, 2018
@jakauppila
Copy link
Contributor

Are the Bento builds forthcoming on vagrantup.com ? I see @tas50 has some deployed.

Thanks a lot for doing this! We were actually planning to start building some ourselves as most of the others out there are pretty out of date.

@cheeseplus
Copy link
Contributor

No, Windows builds right now are hosted by @tas50 while we figure out the build pipelines and make sure there aren't any issues legally.

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

Successfully merging this pull request may close these issues.

3 participants