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 #3630, #3629: Updated the motdConfiguration to the most up to date... #135
Fixes #3630, #3629: Updated the motdConfiguration to the most up to date... #135
Conversation
Added a new commit to separate parameters from logic |
# Here, we will define reports for elements which only have a local visibility (like the MOTD file presence detection). | ||
!windows.rudder_motd_absent:: | ||
"any" | ||
usebundle => rudder_common_report("${service_name}", "log_repaired", "&TRACKINGKEY&", "MOTD Configuration", "None", "The ${service_name} file was absent, it will be created"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should not have StringTemplate variable in this file; the goal is really to have pure cfengine
Pr updated, moved TRACKINGKEY to the parameters array |
bundle agent rudder_motd_configuration_reporting(class_prefix, service_name) | ||
{ | ||
methods: | ||
"any" usebundle => rudder_common_reports_generic("${service_name}", "${class_prefix}_file_edition", "${rudder_motd_trackingkey}", "MOTD Configuration", "None", "The ${service_name} file"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
where is ${rudder_motd_trackingkey} defined ?
…ate Technique creation Best practices.
…ate Technique creation Best practices. (Separated parameters from logic)
…ate Technique creation Best practices. (Removed TRACKINGVARIABLE entry from metadata.xml)
…configuration_with_best_practices Fixes #3630, #3629: Updated the motdConfiguration to the most up to date...
…hich_prevent_to_relaunch_cfe Fixes #3996 - Fix typo into /opt/rudder/bin/check-rudder-agent which prevent from relaunching Rudder agent
... Technique creation Best practices.
Ticket: http://www.rudder-project.org/redmine/issues/3630
To be merged in the 2.6 branch.