Skip to content

Conversation

@jan-cerny
Copy link
Collaborator

Rewrite the remediation by using specialized modules instead of the shell module. Consequently, the remediation becomes idempotent.

Resolves: https://issues.redhat.com/browse/OPENSCAP-6249

Review Hints:

  • ./build_product --playbook-per-rule rhel9
  • manually replace hosts by hosts: all in build/rhel9/playbooks/cis/accounts_user_dot_group_ownership.yml
  • run ansible-playbook -u root -i YOUR_IP, build/rhel9/playbooks/cis/accounts_user_dot_group_ownership.yml at least twice and compare the output of the first run with the second run and so on, verify that the second and next runs don't change anything and that the output contains only "ok" or "skipping"
  • apart from that, run automatus Tss with --remediate-using ansible

Rewrite the remediation by using specialized modules instead of the
shell module. Consequently, the remediation becomes idempotent.

Resolves: https://issues.redhat.com/browse/OPENSCAP-6249
@jan-cerny jan-cerny added this to the 0.1.79 milestone Sep 26, 2025
@jan-cerny jan-cerny added the Ansible Ansible remediation update. label Sep 26, 2025
@openshift-ci
Copy link

openshift-ci bot commented Sep 26, 2025

@jan-cerny: The following test 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/e2e-aws-openshift-node-compliance c0b7107 link true /test e2e-aws-openshift-node-compliance

Full PR test history. Your PR dashboard.

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.

@Mab879 Mab879 self-assigned this Sep 26, 2025
@Mab879 Mab879 merged commit 0f1abde into ComplianceAsCode:master Sep 26, 2025
135 of 138 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