Skip to content

Fix glob expansion order in sshd_lineinfile test scripts for RHEL8#14899

Merged
Mab879 merged 1 commit into
ComplianceAsCode:masterfrom
Arden97:sshd_weekly_fix
Jul 16, 2026
Merged

Fix glob expansion order in sshd_lineinfile test scripts for RHEL8#14899
Mab879 merged 1 commit into
ComplianceAsCode:masterfrom
Arden97:sshd_weekly_fix

Conversation

@Arden97

@Arden97 Arden97 commented Jul 16, 2026

Copy link
Copy Markdown
Member

Description:

Rationale:

  • On systems where /etc/ssh/sshd_config.d/ is not pre-created (RHEL 8), bash keeps the unexpanded literal /etc/ssh/sshd_config.d/* in the array
  • When grep later tries to open that literal path, it fails with exit code 2
  • 2 test scenarios for sshd_lineinfile template were updated

@openshift-ci openshift-ci Bot added the do-not-merge/work-in-progress Used by openshift-ci bot. label Jul 16, 2026
@openshift-ci

openshift-ci Bot commented Jul 16, 2026

Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@Arden97 Arden97 added this to the 0.1.82 milestone Jul 16, 2026
@Arden97
Arden97 marked this pull request as ready for review July 16, 2026 15:46
@openshift-ci openshift-ci Bot removed the do-not-merge/work-in-progress Used by openshift-ci bot. label Jul 16, 2026
@openshift-ci

openshift-ci Bot commented Jul 16, 2026

Copy link
Copy Markdown

@Arden97: 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-node-compliance 8216efa link true /test e2e-aws-openshift-node-compliance
ci/prow/e2e-aws-openshift-platform-compliance 8216efa link true /test e2e-aws-openshift-platform-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.

@Mab879 Mab879 self-assigned this Jul 16, 2026
@Mab879
Mab879 merged commit 71f1202 into ComplianceAsCode:master Jul 16, 2026
70 of 73 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants