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 #7375: Add an option in the web interface to completely disable reporting #955
Conversation
val allModes = ComplianceModeName.allModes.map(_.name).mkString("(", "|", ")") | ||
allModes | ||
} | ||
, errorMsg = s"Forbiden value, only 'full-compliance' and 'changes-only' are authorized" |
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.
the error message seems incomplete - reports-disabled should be authorized
PR updated |
3199e0e
to
244ee60
Compare
|
||
object ChangesOnly { | ||
final case object ChangesOnly extends ComplianceModeName { |
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.
no need for final on object, they are already (you can look at the bytecode for that, or it's certainly specified somewhere in some scala spec). But you can let it here, I does no arm.
PR updated |
244ee60
to
2b8e5a2
Compare
Still some things to do (last @fanf remark) And also waiting new text about the new mode in the webapp should be ok verrryyyy soon |
PR updated |
2b8e5a2
to
1b3f4c5
Compare
PR updated |
84bd20c
to
d6a4765
Compare
PR updated |
@@ -12,7 +12,11 @@ | |||
network traffic, but provides more precise reporting and may be necessary to prove compliance in your organization.<br/> | |||
In <b>Changes only</b> mode, reports will only be sent when the agent makes a change or an error occurs on a node | |||
(these are 'repair' or 'error' reports). This mode saves a lot of log space and bandwidth, but leads to some assumptions | |||
about actual configuration status in reporting. | |||
about actual configuration status in reporting.<br/> | |||
In <b>Disabled</b>b> mode, no reports will be sent, and rudder-agent will not re-configure the local syslog to send reports. |
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.
There is a b> that should not be here
PR updated |
d6a4765
to
bfdb3e2
Compare
PR updated |
bfdb3e2
to
86b01b0
Compare
PR updated |
86b01b0
to
2b46102
Compare
…the_web_interface_to_completely_disable_reporting Fixes #7375: Add an option in the web interface to completely disable reporting
https://www.rudder-project.org/redmine/issues/7375