Skip to content

[fix] Fix ssh version integration test#362

Merged
EthanHeilman merged 7 commits into
openpubkey:mainfrom
EthanHeilman:fix-ssh-ver-test
Oct 8, 2025
Merged

[fix] Fix ssh version integration test#362
EthanHeilman merged 7 commits into
openpubkey:mainfrom
EthanHeilman:fix-ssh-ver-test

Conversation

@EthanHeilman
Copy link
Copy Markdown
Member

@EthanHeilman EthanHeilman commented Oct 8, 2025

Integration test is currently breaking due to changes to the docker images in newer versions of RHEL and Arch Linux.

This PR:

  • Fixes the error output bug where the error is not displayed
  • Fixes the issue in RHEL and Arch linux
2025-10-08T14:10:26.6864294Z     openssh_version_test.go:115: Running setup command: pacman -Sy --noconfirm openssh sed
2025-10-08T14:11:25.7627478Z     openssh_version_test.go:118: 
2025-10-08T14:11:25.7629066Z         	Error Trace:	/home/runner/work/opkssh/opkssh/test/integration/openssh_version_test.go:118
2025-10-08T14:11:25.7631835Z         	            				/home/runner/work/opkssh/opkssh/test/integration/openssh_version_test.go:85
2025-10-08T14:11:25.7632741Z         	Error:      	Not equal: 
2025-10-08T14:11:25.7633339Z         	            	expected: 0
2025-10-08T14:11:25.7633973Z         	            	actual  : 1
2025-10-08T14:11:25.7634749Z         	Test:       	TestOpenSSHVersionDetection/Arch_Linux
2025-10-08T14:11:25.7653777Z         	Messages:   	setup command failed with output: &{� ... %!s(*net.UnixConn=&{{0xc000371380}}) %!s(int=0) %!s(int=0) <nil> %!s(int=-1) %!s(int=-1)}
2025-10-08T14:11:25.7656861Z 2025/10/08 14:11:25 🐳 Stopping container: 327e628a3ed4
2025-10-08T14:11:35.8471836Z 2025/10/08 14:11:35 ✅ Container stopped: 327e628a3ed4
2025-10-08T14:11:35.8472983Z 2025/10/08 14:11:35 🐳 Terminating container: 327e628a3ed4
2025-10-08T14:11:35.8497344Z 2025/10/08 14:11:35 🚫 Container terminated: 327e628a3ed4
2025-10-08T14:11:35.8499387Z     openssh_version_test.go:109: Warning: failed to terminate container: Error response from daemon: removal of container 327e628a3ed47c253457134402ee8f94b93479f4c6e434de0c4a9bc875b34c1d is already in progress
2025-10-08T14:11:35.8501112Z --- FAIL: TestOpenSSHVersionDetection (157.08s)
2025-10-08T14:11:35.8501840Z     --- PASS: TestOpenSSHVersionDetection/Debian/Ubuntu (18.20s)
2025-10-08T14:11:35.8502609Z     --- FAIL: TestOpenSSHVersionDetection/RHEL/CentOS (26.90s)
2025-10-08T14:11:35.8503292Z     --- PASS: TestOpenSSHVersionDetection/SUSE (24.78s)
2025-10-08T14:11:35.8503980Z     --- FAIL: TestOpenSSHVersionDetection/Arch_Linux (87.20s)

@EthanHeilman EthanHeilman added the bug Something isn't working label Oct 8, 2025
@EthanHeilman EthanHeilman changed the title Fix ssh version integration test [fix] Fix ssh version integration test Oct 8, 2025
@EthanHeilman EthanHeilman merged commit 4af397e into openpubkey:main Oct 8, 2025
16 checks passed
renovate Bot added a commit to sdwilsh/ansible-playbooks that referenced this pull request Jan 5, 2026
@renovate
chore(deps): update dependency openpubkey/opkssh to v0.11.0
e5d091e 
##### [\`v0.11.0\`](https://github.com/openpubkey/opkssh/releases/tag/v0.11.0)

##### 🚀 Features

- Add support for custom group claims [@mvanderlee](https://github.com/mvanderlee) ([#133](openpubkey/opkssh#133))
- feat: Flag to print SSH cert and private key rather than FS [@EthanHeilman](https://github.com/EthanHeilman) ([#437](openpubkey/opkssh#437))
- feat: Process extra arguments to the verify command [@justincmoy](https://github.com/justincmoy) ([#436](openpubkey/opkssh#436))
- Add warning message when email claim is missing from ID token @[copilot-swe-agent\[bot\]](https://github.com/apps/copilot-swe-agent) ([#374](openpubkey/opkssh#374))
- \[feat] Add new "inspect" subcommand [@stmcginnis](https://github.com/stmcginnis) ([#349](openpubkey/opkssh#349))
- Add CLI reference documentation [@stmcginnis](https://github.com/stmcginnis) ([#365](openpubkey/opkssh#365))
- Include signature JSON in `inspect` output [@stmcginnis](https://github.com/stmcginnis) ([#358](openpubkey/opkssh#358))
- - docs: Add Amazon Cognito as tested provider [@Foorack](https://github.com/Foorack) ([#414](openpubkey/opkssh#414))
- docs: Add documentation for opkssh and sssd integration [@vigneshmanick](https://github.com/vigneshmanick) ([#409](openpubkey/opkssh#409))
- Added SELinux support for sudo logging [@descensus](https://github.com/descensus) ([#376](openpubkey/opkssh#376))
- Update CLI documentation @[github-actions\[bot\]](https://github.com/apps/github-actions) ([#368](openpubkey/opkssh#368))

##### 🐛 Bug Fixes

- Fix race condition in ReadHome [@gcorrall](https://github.com/gcorrall) ([#391](openpubkey/opkssh#391))
- \[fix] Use lowercase for positional argument placeholders [@t38miwa](https://github.com/t38miwa) ([#361](openpubkey/opkssh#361))
- fix typo in commands/verify.go [@DevRockstarZ](https://github.com/DevRockstarZ) ([#336](openpubkey/opkssh#336))
- Doc: fix small errors in policy plugin doc [@PotatoesMaster](https://github.com/PotatoesMaster) ([#344](openpubkey/opkssh#344))
- Correct macOS name [@stmcginnis](https://github.com/stmcginnis) ([#341](openpubkey/opkssh#341))

##### 🧰 Maintenance

- chore: document upstream nix usage & remove nix flake [@datosh](https://github.com/datosh) ([#383](openpubkey/opkssh#383))
- Update CLI documentation @[github-actions\[bot\]](https://github.com/apps/github-actions) ([#438](openpubkey/opkssh#438))
- Stop hash pinning docker images [@EthanHeilman](https://github.com/EthanHeilman) ([#421](openpubkey/opkssh#421))
- \[fix] Fix ssh version integration test [@EthanHeilman](https://github.com/EthanHeilman) ([#362](openpubkey/opkssh#362))
- Fix integration tests failing due to pacman keys [@EthanHeilman](https://github.com/EthanHeilman) ([#432](openpubkey/opkssh#432))
- fix(deps): Update docker/setup-buildx-action action to v3.12.0 @[renovate\[bot\]](https://github.com/apps/renovate) ([#426](openpubkey/opkssh#426))
- fix(deps): Update zizmorcore/zizmor-action action to v0.3.0 @[renovate\[bot\]](https://github.com/apps/renovate) ([#413](openpubkey/opkssh#413))
- fix(deps): Update peter-evans/create-pull-request action to v8 @[renovate\[bot\]](https://github.com/apps/renovate) ([#418](openpubkey/opkssh#418))
- fix(deps): Update zizmorcore/zizmor-action action to v0.2.0 @[renovate\[bot\]](https://github.com/apps/renovate) ([#335](openpubkey/opkssh#335))
- fix(deps): Update peter-evans/create-pull-request action to v7.0.9 @[renovate\[bot\]](https://github.com/apps/renovate) ([#407](openpubkey/opkssh#407))
sdwilsh pushed a commit to sdwilsh/ansible-playbooks that referenced this pull request Jan 13, 2026
@renovate @sdwilsh
chore(deps): update dependency openpubkey/opkssh to v0.11.0
a7acc4b 
##### [\`v0.11.0\`](https://github.com/openpubkey/opkssh/releases/tag/v0.11.0)

##### 🚀 Features

- Add support for custom group claims [@mvanderlee](https://github.com/mvanderlee) ([#133](openpubkey/opkssh#133))
- feat: Flag to print SSH cert and private key rather than FS [@EthanHeilman](https://github.com/EthanHeilman) ([#437](openpubkey/opkssh#437))
- feat: Process extra arguments to the verify command [@justincmoy](https://github.com/justincmoy) ([#436](openpubkey/opkssh#436))
- Add warning message when email claim is missing from ID token @[copilot-swe-agent\[bot\]](https://github.com/apps/copilot-swe-agent) ([#374](openpubkey/opkssh#374))
- \[feat] Add new "inspect" subcommand [@stmcginnis](https://github.com/stmcginnis) ([#349](openpubkey/opkssh#349))
- Add CLI reference documentation [@stmcginnis](https://github.com/stmcginnis) ([#365](openpubkey/opkssh#365))
- Include signature JSON in `inspect` output [@stmcginnis](https://github.com/stmcginnis) ([#358](openpubkey/opkssh#358))
- - docs: Add Amazon Cognito as tested provider [@Foorack](https://github.com/Foorack) ([#414](openpubkey/opkssh#414))
- docs: Add documentation for opkssh and sssd integration [@vigneshmanick](https://github.com/vigneshmanick) ([#409](openpubkey/opkssh#409))
- Added SELinux support for sudo logging [@descensus](https://github.com/descensus) ([#376](openpubkey/opkssh#376))
- Update CLI documentation @[github-actions\[bot\]](https://github.com/apps/github-actions) ([#368](openpubkey/opkssh#368))

##### 🐛 Bug Fixes

- Fix race condition in ReadHome [@gcorrall](https://github.com/gcorrall) ([#391](openpubkey/opkssh#391))
- \[fix] Use lowercase for positional argument placeholders [@t38miwa](https://github.com/t38miwa) ([#361](openpubkey/opkssh#361))
- fix typo in commands/verify.go [@DevRockstarZ](https://github.com/DevRockstarZ) ([#336](openpubkey/opkssh#336))
- Doc: fix small errors in policy plugin doc [@PotatoesMaster](https://github.com/PotatoesMaster) ([#344](openpubkey/opkssh#344))
- Correct macOS name [@stmcginnis](https://github.com/stmcginnis) ([#341](openpubkey/opkssh#341))

##### 🧰 Maintenance

- chore: document upstream nix usage & remove nix flake [@datosh](https://github.com/datosh) ([#383](openpubkey/opkssh#383))
- Update CLI documentation @[github-actions\[bot\]](https://github.com/apps/github-actions) ([#438](openpubkey/opkssh#438))
- Stop hash pinning docker images [@EthanHeilman](https://github.com/EthanHeilman) ([#421](openpubkey/opkssh#421))
- \[fix] Fix ssh version integration test [@EthanHeilman](https://github.com/EthanHeilman) ([#362](openpubkey/opkssh#362))
- Fix integration tests failing due to pacman keys [@EthanHeilman](https://github.com/EthanHeilman) ([#432](openpubkey/opkssh#432))
- fix(deps): Update docker/setup-buildx-action action to v3.12.0 @[renovate\[bot\]](https://github.com/apps/renovate) ([#426](openpubkey/opkssh#426))
- fix(deps): Update zizmorcore/zizmor-action action to v0.3.0 @[renovate\[bot\]](https://github.com/apps/renovate) ([#413](openpubkey/opkssh#413))
- fix(deps): Update peter-evans/create-pull-request action to v8 @[renovate\[bot\]](https://github.com/apps/renovate) ([#418](openpubkey/opkssh#418))
- fix(deps): Update zizmorcore/zizmor-action action to v0.2.0 @[renovate\[bot\]](https://github.com/apps/renovate) ([#335](openpubkey/opkssh#335))
- fix(deps): Update peter-evans/create-pull-request action to v7.0.9 @[renovate\[bot\]](https://github.com/apps/renovate) ([#407](openpubkey/opkssh#407))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@EthanHeilman