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

Fixes #9242: Add new reports types to the rudder logger #468

Conversation

amousset
Copy link
Member

@amousset amousset commented Oct 6, 2016

@amousset
Copy link
Member Author

amousset commented Oct 6, 2016

WIP

@amousset
Copy link
Member Author

amousset commented Oct 6, 2016

BTW we need to rename "changes only" mode!

@amousset
Copy link
Member Author

amousset commented Oct 7, 2016

Commit modified

@amousset amousset force-pushed the ust_9242/add_new_reports_types_to_the_rudder_logger branch from e8d4c31 to 37719d9 Compare October 7, 2016 08:39
@amousset
Copy link
Member Author

amousset commented Oct 7, 2016

I will make the PR for Rudder technique once this one is done.

@@ -251,6 +251,7 @@ bundle agent _rudder_common_report(technique_name, status, identifier, component
strcmp("${status}", "result_repaired") ,
strcmp("${status}", "log_warn") ,
strcmp("${status}", "log_repaired") ,
strcmp("${status}", "audit_noncompliant") ,
Copy link
Member Author

Choose a reason for hiding this comment

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

+audit_error

@amousset
Copy link
Member Author

Commit modified

@amousset amousset force-pushed the ust_9242/add_new_reports_types_to_the_rudder_logger branch from 37719d9 to ef64164 Compare October 10, 2016 08:47
@Normation-Quality-Assistant
Copy link
Contributor

OK, merging this PR

@Normation-Quality-Assistant Normation-Quality-Assistant merged commit ef64164 into Normation:master Oct 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants