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

Habitat build works for all versions, eliminates rake #2301

Merged
merged 1 commit into from
Nov 14, 2017

Conversation

adamleff
Copy link
Contributor

@adamleff adamleff commented Nov 9, 2017

The Habitat plan has been modified to support building from the repo rather than relying on a gem being pushed to RubyGems. This allows us to build current packages at every merge rather than only pushing to Habitat Builder when we promote to stable.

This change also enables Expeditor to perform builds for us and removes the dependency on the rake task as it is no longer needed.

@adamleff adamleff requested a review from a team November 9, 2017 20:41
@adamleff adamleff requested a review from a team as a code owner November 9, 2017 20:41
The Habitat plan has been modified to support building from the repo
rather than relying on a gem being pushed to RubyGems. This allows
us to build current packages at every merge rather than only pushing to
Habitat Builder when we promote to stable.

This change also enables Expeditor to perform builds for us and removes
the dependency on the rake task as it is no longer needed.

Signed-off-by: Adam Leff <[email protected]>
@adamleff adamleff force-pushed the adamleff/habitat-plan branch from cb07d8c to 0672b8c Compare November 9, 2017 20:44
Copy link
Contributor

@schisamo schisamo left a comment

Choose a reason for hiding this comment

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

Left a few comments I am happy to chat with you about in the meatspace.

@@ -10,6 +10,9 @@ docker_images:
build_args:
- GEM_SOURCE: http://artifactory.chef.co/omnibus-gems-local

habitat_packages:
- inspec
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe this needs/should be a key...you might want to optionally export a container to docker hub:

habitat_packages:
  - inspec:
      export:
        - docker

This would export the current build and tag with a version (MAJOR.MINOR.PATCH-YYYYMMDDHHMMSS), channel (unstable). We can then "promote" the image (e.g. update channel tags) along with the omnibus packages via Expeditor artifact_actions.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was going off of the docs which make it seem like it can be a simple string as long as there were no options:

http://expeditor-docs.es.chef.io/configuration/habitat-packages/#habitat_packages

The InSpec docker image is built and consumed in a different way currently that I'd like to change in the future, but for now I need to keep them separate. Expeditor is at least handling the docker stuff for us, so that's cool.

@@ -24,46 +25,63 @@ pkg_build_deps=(
)
Copy link
Contributor

Choose a reason for hiding this comment

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

Does it make sense to use scaffolding-ruby here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Unless something changed recently, the Ruby scaffolding worked well for rails, etc... but not for standalone ruby projects. Happy to consider a change in the future, but I'd like the focus of this PR to be on the automation of the existing package building vs. refactoring the package.

@adamleff
Copy link
Contributor Author

adamleff commented Nov 9, 2017

@schisamo thanks for the review! I replied to the comments and happy to Zoom or continue the conversation here in the PR further, for sure.

@schisamo
Copy link
Contributor

schisamo commented Nov 9, 2017

:shipit:

Copy link
Contributor

@arlimus arlimus left a comment

Choose a reason for hiding this comment

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

Thank you @adamleff !! 👍

@arlimus arlimus merged commit 01b65db into master Nov 14, 2017
@arlimus arlimus deleted the adamleff/habitat-plan branch November 14, 2017 04:01
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