Skip to content

[6.19.z] Fix test_positive_dump_enc_yaml: accept both RSA and Ed25519 SSH keys - #22237

Merged
jnagare-redhat merged 1 commit into
SatelliteQE:6.19.zfrom
jnagare-redhat:cherry-pick-enc-yaml-fix-6.19.z
Jul 24, 2026
Merged

[6.19.z] Fix test_positive_dump_enc_yaml: accept both RSA and Ed25519 SSH keys#22237
jnagare-redhat merged 1 commit into
SatelliteQE:6.19.zfrom
jnagare-redhat:cherry-pick-enc-yaml-fix-6.19.z

Conversation

@jnagare-redhat

Copy link
Copy Markdown
Contributor

Summary

Test plan

  • Run pytest tests/foreman/cli/test_host.py -k test_positive_dump_enc_yaml

🤖 Generated with Claude Code

…SatelliteQE#21896)

Satellite/Foreman has moved from RSA to Ed25519 SSH keys for remote
execution. The test was checking for 'ssh-rsa' which no longer exists
in the ENC dump.

Updated the assertion to accept both 'ssh-rsa' and 'ssh-ed25519' to
support both older and newer Satellite versions.

The ENC dump now contains:
  remote_execution_ssh_keys:
  - ssh-ed25519 AAAAC3NzaC1lZDI1NTE5...

instead of the old RSA format.
@jnagare-redhat jnagare-redhat added No-CherryPick PR doesnt need CherryPick to previous branches 6.19.z labels Jul 23, 2026
@jnagare-redhat

Copy link
Copy Markdown
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_host.py::test_positive_dump_enc_yaml

@Satellite-QE

Copy link
Copy Markdown
Collaborator

PRT Result

Build Number: 16222
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_host.py::test_positive_dump_enc_yaml --external-logging
Test Result : ================== 1 passed, 5 warnings in 934.32s (0:15:34) ===================

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jul 23, 2026
@jnagare-redhat
jnagare-redhat merged commit 3a0988a into SatelliteQE:6.19.z Jul 24, 2026
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.19.z No-CherryPick PR doesnt need CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants