Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for specifying SSH key type to be generated #583

Merged
merged 2 commits into from
Dec 13, 2022

Conversation

bdwyertech
Copy link
Contributor

@bdwyertech bdwyertech commented May 2, 2022

Description

AL2022+ is based on Fedora 35 which only supports ed25519 out of the box

This exposes the key type passed to the AWS API as optional parameter, aws_ssh_key_type.

Packer added support for same via hashicorp/packer-plugin-amazon#179

Issues Resolved

Type of Change

Our release process assumes you are using Conventional Commit messages.

The most important prefixes you should have in mind are:

  • _fix_: which represents bug fixes, and correlates to a SemVer patch.
  • _feat_: which represents a new feature, and correlates to a SemVer minor.
  • _feat!_:, or fix!:, refactor!:, etc., which represent a breaking change (indicated by the !) and will result in a major version change.

If you have not included a conventional commit message this can be fixed on merge.

Check List

…ated

AL2022+ is based on Fedora 35 which only supports ed25519 out of the box
KSerrania added a commit to DataDog/datadog-agent that referenced this pull request Jun 2, 2022
Adds Ubuntu 22.04 x64 tests to kitchen.

Modifies the test suites generation to automatically detect whether an rsa, ed25519, or the default key created by kitchen, should be used, depending on the target platforms.

Renames some EC2 driver variables for consistency.

Note: Ubuntu 22.04 arm64 tests can be added once test-kitchen/kitchen-ec2#583 is merged and released (to support ed25519 key creation).
@tas50 tas50 merged commit 2cdf999 into test-kitchen:main Dec 13, 2022
@tas50
Copy link
Member

tas50 commented Dec 13, 2022

@bdwyertech Can you make sure to get this into the docs in the main TK repo?

@bdwyertech
Copy link
Contributor Author

@bdwyertech Can you make sure to get this into the docs in the main TK repo?

Done @tas50

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