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
Merge custom and bundled compliance benchmarks #409
Conversation
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.
This pull request does not contain a valid label. Please add one of the following labels: bug, enhancement, documentation
ca8b911
to
0ca3e36
Compare
Codecov Report
@@ Coverage Diff @@
## main #409 +/- ##
=======================================
Coverage 66.27% 66.27%
=======================================
Files 3 3
Lines 86 86
=======================================
Hits 57 57
Misses 18 18
Partials 11 11
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
0ca3e36
to
1e4b647
Compare
1e4b647
to
c4cfb3b
Compare
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.
I added a small comment. just to confirm the behavior
c4cfb3b
to
fbd6559
Compare
What does this PR do?
Instead of replacing the /etc/datadog-agent/compliance.d directory with the content
of a configmap, merge the benchmarks bundled with the agent and the one provided
with the configmap.
Motivation
This behaviour is the same than the security policies and it allows to both add new benchmarks
and replace existing ones.
Additional Notes
Anything else we should know when reviewing?
Describe your test plan
Write there any instructions and details you may have to test your PR.