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

[WIP] Box build pipeline #372

Closed
wants to merge 5 commits into from
Closed

[WIP] Box build pipeline #372

wants to merge 5 commits into from

Conversation

yzl
Copy link

@yzl yzl commented Jun 13, 2015

cc @fnichol @schisamo @tyler-ball

Build a set of boxes based on the most recently changed files in git, upload the box artifacts to atlas, and release them.

  • Tested against templates for public boxen.
  • Support proprietary/private boxen.
  • The Rakefile is dreadful because rake makes me feel like I have more than 8 thumbs. Suggestions on how to make it better would be welcome.

require 'json'
require 'net/http'

# Enables `bundle exec bundle exec rake do_all[ubuntu-12.04-amd64,centos-7.1-x86_64]

Choose a reason for hiding this comment

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

(nit) bundle exec twice

@schisamo
Copy link
Contributor

Just an FYI that first-class WinRM support is now part of Packer core:
https://twitter.com/mitchellh/status/610167963673473024
hashicorp/packer#451

This should make the Windows base box builds just work.

@cheeseplus
Copy link
Contributor

Given the additional work I've done to validate this approach I'm 👍 on merging this and then I'll apply my changes on top. This is more than good enough for an MVP and we can lament or toast our choice of Rake in hindsight.

@fnichol
Copy link
Contributor

fnichol commented Jul 10, 2015

Woo 👍

@cheeseplus cheeseplus mentioned this pull request Jul 10, 2015
@cheeseplus cheeseplus self-assigned this Jul 10, 2015
@fnichol fnichol closed this in #398 Jul 10, 2015
@fnichol fnichol removed the Reviewing label Jul 10, 2015
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.

5 participants