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

Integrate with Chef Compliance #124

Merged
merged 4 commits into from Oct 27, 2016

Conversation

Projects
None yet
3 participants
@chris-rock
Collaborator

chris-rock commented Oct 27, 2016

Description

This PR brings back the Chef Compliance integration by using the new Chef Handler architecture. It also leverages the inspec compliance fetcher. Also We save time by running inspec only once instead for each profile individually.

  • Basic Integration
  • Allow reporting of multiple profiles with one inspec exec run

@binamov binamov added the in progress label Oct 27, 2016

@chris-rock chris-rock added this to the 2.0 milestone Oct 27, 2016

@chris-rock chris-rock changed the title from WIP: Integrate with Chef Compliance to Integrate with Chef Compliance Oct 27, 2016

vjeffrey and others added some commits Oct 19, 2016

Clean up handler directory after executing
Signed-off-by: Victoria Jeffrey <vjeffrey@chef.io>
support chef compliance
Signed-off-by: Christoph Hartmann <chris@lollyrock.com>
@chris-rock

This comment has been minimized.

Collaborator

chris-rock commented Oct 27, 2016

This PR is not touching the unit tests yet. This is covered in #128

@vjeffrey

This comment has been minimized.

Collaborator

vjeffrey commented Oct 27, 2016

🌟 i just tested it, works great. will look thru code in a minute, just need to finish making lunch.

@vjeffrey

This comment has been minimized.

Collaborator

vjeffrey commented Oct 27, 2016

👍 🎉

@chris-rock

This comment has been minimized.

Collaborator

chris-rock commented Oct 27, 2016

Thanks for review @vjeffrey

@chris-rock chris-rock merged commit 2d769f4 into master Oct 27, 2016

2 checks passed

DCO This commit has a DCO Signed-off-by
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@chris-rock chris-rock deleted the vj/integrate-with-compliance branch Oct 27, 2016

@binamov binamov removed the in progress label Oct 27, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment