Skip to content

Make sure ansible task is properly executed when no variable is defined#13970

Merged
jan-cerny merged 1 commit into
ComplianceAsCode:masterfrom
ggbecker:fix-ansible-task
Oct 3, 2025
Merged

Make sure ansible task is properly executed when no variable is defined#13970
jan-cerny merged 1 commit into
ComplianceAsCode:masterfrom
ggbecker:fix-ansible-task

Conversation

@ggbecker
Copy link
Copy Markdown
Member

@ggbecker ggbecker commented Oct 2, 2025

Description:

  • Make sure ansible task is properly executed when no variable is defined
    • There might be cases where interactive_user_homes is not defined so we need to be sure there is a default value.

Rationale:

  • Fix:
    • 2025-10-02 11:18:18 test.py:49: lib.results.report_plain:238: ERROR playbook: Ensure the Default Umask is Set Correctly For Interactive Users - Find dot files in interactive user home directories ({"msg": "'interactive_user_homes' is undefined. 'interactive_user_homes' is undefined"})

@ggbecker ggbecker added this to the 0.1.79 milestone Oct 2, 2025
@ggbecker ggbecker requested a review from jan-cerny October 2, 2025 11:34
@ggbecker ggbecker added bugfix Fixes to reported bugs. Ansible Ansible remediation update. labels Oct 2, 2025
@jan-cerny jan-cerny self-assigned this Oct 2, 2025
There might be cases where interactive_user_homes is not defined so we
need to be sure there is a default value.
@ggbecker ggbecker requested a review from jan-cerny October 2, 2025 13:11
@jan-cerny
Copy link
Copy Markdown
Collaborator

/retest

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Oct 2, 2025

@ggbecker: 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 391a065 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.

@jan-cerny jan-cerny merged commit 940a586 into ComplianceAsCode:master Oct 3, 2025
134 of 136 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. bugfix Fixes to reported bugs.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants