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

Add insecure flag for `Collector::ChefVisibility` #153

Merged
merged 1 commit into from Nov 9, 2016

Conversation

Projects
None yet
3 participants
@jerryaldrichiii
Collaborator

jerryaldrichiii commented Nov 8, 2016

Description

This adds support for node['audit']['insecure'] to the Chef Visibility collector

Issues Resolved

#150

Check List

Add insecure flag for `Collector::ChefVisibility`
Signed-off-by: Jerry Aldrich III <jerry@chef.io>
@vjeffrey

👍 tested locally with default insecure and insecure as true; both work great. Thanks for this change @jerryaldrichiii -- this looks like a great model to expand out to chef-compliance and chef-server

@chris-rock

This comment has been minimized.

Collaborator

chris-rock commented Nov 9, 2016

We merge this for now, lets think about integration tests with visibility

@chris-rock chris-rock merged commit 66521d2 into chef-cookbooks:master Nov 9, 2016

2 checks passed

DCO This commit has a DCO Signed-off-by
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment