CMP-4439: Add manual rule kubevirt-restrict-update-access-to-cdi-cr (CIS OCP-Virt 1.14) - #14939
Conversation
|
Hi @taimurhafeez. Thanks for your PR. I'm waiting for a ComplianceAsCode member to verify that this patch is reasonable to test. If it is, they should reply with Tip We noticed you've done this a few times! Consider joining the org to skip this step and gain Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
LGTM on the rule content. Two asks: (1) rebase to drop the duplicated build-script/docs change once #14920 merges; (2) use the fully-qualified |
rhmdnd
left a comment
rhmdnd
left a comment
There was a problem hiding this comment.
Looks good, just need to rebase and then update the description so that users know they can use CustomRules to automate this.
249ae34 to
daa2a49
Compare
openshift-ci
Bot
added
needs-rebase
daa2a49 to
92a11b9
Compare
…CIS OCP-Virt 1.14)
…iption that explains manual status and points organizations to CustomRules for automation. Co-Authored-By: Claude Opus 4.6 (1M context)
92a11b9 to
eadbae5
Compare
|
/ok-to-test |
openshift-ci
Bot
added
ok-to-test
|
/retest-required |
|
/retest |
|
@taimurhafeez: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
5 participants
Add a manual rule for CIS OCP-Virt control 1.14 "Restrict update access to the CDI CR". The rule documents the audit procedure (oc adm policy who-can update cdi) and is added to the cis-vm-extension CEL profile. Also relax build_cel_content.py to warn instead of error when a CEL profile references rules without CEL checks, allowing manual rules in CEL profiles. Update CEL documentation accordingly
Co-Authored-By: Claude