-
Notifications
You must be signed in to change notification settings - Fork 685
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
OCP: Added control resonse for NIST AU #8511
Conversation
This datastream diff is auto generated by the check Click here to see the full diffPlatform #ocp4.8 not defined in platform specification
Platform #ocp4.8 not defined in platform specification
Platform #ocp4.6 not defined in platform specification
Platform #ocp4.6 not defined in platform specification |
/retest |
8b00a09
to
88c30de
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.
Some nits and questions inline, overall looking good.
applications/openshift/general/alert_receiver_configured/rule.yml
Outdated
Show resolved
Hide resolved
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.
/lgtm
We are going to have a new rule to check if cluster logging operator has been installed, that requires api-resource-collector to have additional permission to read clusterloggings resources.[1] PR to the new rule: ComplianceAsCode/content#8511
/retest |
@Vincent056 please rebase |
...penshift/logging/cluster_logging_operator_exist/tests/does_not_have_logging_instance.fail.sh
Outdated
Show resolved
Hide resolved
...openshift/logging/cluster_logging_operator_exist/tests/have_cluster_logging_instance.pass.sh
Outdated
Show resolved
Hide resolved
749f5cc
to
fac6ca9
Compare
Added control response for AU-9(3) AU-5(1) AU-5(2) AU-12(1) AU-12(3) AU-3(2)
e2e:
which looks like a transient failure. The moderate suite didn't run because the cluster didn't install. |
/retest |
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.
/lgtm
Added control response for AU-9(3) AU-5(1) AU-5(2) AU-12(1) AU-12(3) AU-3(2)
Add rule:
alert_receiver_configured
cluster_logging_operator_exist