Skip to content

Rework Ansible remediation in accounts_umask_interactive_users#13934

Merged
ggbecker merged 1 commit into
ComplianceAsCode:masterfrom
jan-cerny:accounts_umask_interactive_users
Sep 29, 2025
Merged

Rework Ansible remediation in accounts_umask_interactive_users#13934
ggbecker merged 1 commit into
ComplianceAsCode:masterfrom
jan-cerny:accounts_umask_interactive_users

Conversation

@jan-cerny
Copy link
Copy Markdown
Collaborator

Rewrite the remediation that is currently using a shell command to use specialized Ansible modules instead.

The reason for this change is that the task using the shell command isn't idempotent.

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

Rewrite the remediation that is currently using a shell command
to use specialized Ansible modules instead.

The reason for this change is that the task using the shell command
isn't idempotent.

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

openshift-ci Bot commented Sep 25, 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/e2e-aws-openshift-platform-compliance 24bf884 link true /test e2e-aws-openshift-platform-compliance
ci/prow/e2e-aws-openshift-node-compliance 24bf884 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.

@ggbecker ggbecker self-assigned this Sep 29, 2025
Copy link
Copy Markdown
Member

@ggbecker ggbecker left a comment

Choose a reason for hiding this comment

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

I have run automatus and all tests pass.

I also changed the system and performed the ansible remediation manually and only during the first execution the system changed.

@ggbecker ggbecker merged commit f1dc8ab into ComplianceAsCode:master Sep 29, 2025
133 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