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

Add ChefDeprecations/EOLAuditModeUsage #344

Merged
merged 1 commit into from
Oct 23, 2019
Merged

Add ChefDeprecations/EOLAuditModeUsage #344

merged 1 commit into from
Oct 23, 2019

Conversation

tas50
Copy link
Contributor

@tas50 tas50 commented Oct 23, 2019

Detect the legacy audit mode and suggest folks check out InSpec
instead. This also cleans up some bad copyright dates.

33 occurrences of this were detected on Supermarket

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

@tas50 tas50 requested review from a team as code owners October 23, 2019 20:42
@tas50
Copy link
Contributor Author

tas50 commented Oct 23, 2019

Resolves #336

Detect the legacy audit mode and suggest folks check out InSpec
instead. This also cleans up some bad copyright dates.

Signed-off-by: Tim Smith <tsmith@chef.io>
Copy link
Contributor

@tyler-ball tyler-ball left a comment

Choose a reason for hiding this comment

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

I poked around on docs.chef.io to see if we had information about converting existing control_groups to using the audit cookbook but did not see anything. It would be nice if we had a resource like that for users rather than just sending them to the inspec home page, but without that I think this is ready to merge 👍

@tas50
Copy link
Contributor Author

tas50 commented Oct 23, 2019

I asked product where we should send folks and inspec.io and the examples pages came up. We don't have a migration page, which would be super useful to link to.

@tas50 tas50 merged commit b344e63 into master Oct 23, 2019
@chef-expeditor chef-expeditor bot deleted the audit_no_more branch October 23, 2019 21:02
@tyler-ball tyler-ball mentioned this pull request Oct 23, 2019
7 tasks
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.

2 participants