Skip to content

Tests downstream without ssh keygen#6612

Merged
blackboxsw merged 2 commits into
canonical:mainfrom
blackboxsw:tests-downstream-without-ssh-keygen
Dec 12, 2025
Merged

Tests downstream without ssh keygen#6612
blackboxsw merged 2 commits into
canonical:mainfrom
blackboxsw:tests-downstream-without-ssh-keygen

Conversation

@blackboxsw
Copy link
Copy Markdown
Collaborator

Proposed Commit Message

    tests: skip azure ssh-keygen unittests when ssh-keygen not installed
    
    Followup to baedc33ac to support skipping azure ssh-keygen related
    tests from running in environments without ssh-keygen installed.

Additional Context

Daily recipe builds fail for resolute because the default images created do not have ssh-keygen installed
https://launchpadlibrarian.net/836531696/buildlog_ubuntu-resolute-amd64.cloud-init_100.0.0.daily-202512092133-78a748808~ubuntu26.04.1_BUILDING.txt.gz

                if err_filename is not None:
>                   raise child_exception_type(errno_num, err_msg, err_filename)
E                   FileNotFoundError: [Errno 2] No such file or directory: b'ssh-keygen'

/usr/lib/python3.13/subprocess.py:1972: FileNotFoundError

Test Steps

Merge type

  • Squash merge using "Proposed Commit Message"
  • Rebase and merge unique commits. Requires commit messages per-commit each referencing the pull request number (#<PR_NUM>)

@github-actions github-actions Bot added the packaging Supplemental package review requested label Dec 9, 2025
@blackboxsw blackboxsw requested a review from holmanb December 9, 2025 21:55
Followup to baedc33 to support skipping azure ssh-keygen related
tests from running in environments without ssh-keygen installed.
@blackboxsw blackboxsw force-pushed the tests-downstream-without-ssh-keygen branch from 1163d96 to 713f1d7 Compare December 9, 2025 22:01
Copy link
Copy Markdown
Member

@holmanb holmanb left a comment

Choose a reason for hiding this comment

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

Approved once CI passes. Thanks - and agreed. Even if we end up adding this as a test dependency it's still better to keep unit tests passing independent of system binaries.

@blackboxsw blackboxsw merged commit e6955e6 into canonical:main Dec 12, 2025
21 checks passed
holmanb pushed a commit that referenced this pull request Dec 18, 2025
…6612)

Followup to baedc33 to support skipping azure ssh-keygen related
tests from running in environments without ssh-keygen installed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

packaging Supplemental package review requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants