Skip to content

Fix Ansible sysctl template#14161

Merged
Mab879 merged 1 commit intoComplianceAsCode:masterfrom
jan-cerny:master_kernel_core_pattern
Nov 21, 2025
Merged

Fix Ansible sysctl template#14161
Mab879 merged 1 commit intoComplianceAsCode:masterfrom
jan-cerny:master_kernel_core_pattern

Conversation

@jan-cerny
Copy link
Collaborator

If the value of the sysctl item contains a special character it needs to be escaped before using it in a regular expression or the regular expression won't work correctly. This happens in rule sysctl_kernel_core_pattern where the value is |/bin/false which contains the | character.

This fixes multiple failing test scenarios for Ansible remediation in rule sysctl_kernel_core_pattern.

If the value of the sysctl item contains a special character
it needs to be escaped before using it in a regular expression
or the regular expression won't work correctly. This happens
in rule sysctl_kernel_core_pattern where the value is `|/bin/false`
which contains the `|` character.

This fixes multiple failing test scenarios for Ansible remediation
in rule sysctl_kernel_core_pattern.
@jan-cerny jan-cerny added this to the 0.1.80 milestone Nov 21, 2025
@jan-cerny jan-cerny added the Ansible Ansible remediation update. label Nov 21, 2025
@Mab879 Mab879 self-assigned this Nov 21, 2025
@jan-cerny jan-cerny modified the milestones: 0.1.80, 0.1.79 Nov 21, 2025
@openshift-ci
Copy link

openshift-ci bot commented Nov 21, 2025

@jan-cerny: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/4.14-images 0212e7a link true /test 4.14-images
ci/prow/4.19-images 0212e7a link true /test 4.19-images
ci/prow/4.17-images 0212e7a link true /test 4.17-images
ci/prow/4.16-images 0212e7a link true /test 4.16-images
ci/prow/4.12-images 0212e7a link true /test 4.12-images
ci/prow/4.18-images 0212e7a link true /test 4.18-images
ci/prow/images 0212e7a link true /test images
ci/prow/e2e-aws-openshift-platform-compliance 0212e7a link true /test e2e-aws-openshift-platform-compliance
ci/prow/e2e-aws-openshift-node-compliance 0212e7a link true /test e2e-aws-openshift-node-compliance

Full PR test history. Your PR dashboard.

Details

Instructions 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.

Copy link
Member

@Mab879 Mab879 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automatus tests pass locally.

@Mab879
Copy link
Member

Mab879 commented Nov 21, 2025

/retest-failed

@Mab879 Mab879 merged commit d48f677 into ComplianceAsCode:master Nov 21, 2025
131 of 140 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ansible Ansible remediation update.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants