Skip to content

Extend EC2 joining method to other AWS regions#39045

Merged
tigrato merged 3 commits intomasterfrom
tigrato/aws-missing-certs
Mar 7, 2024
Merged

Extend EC2 joining method to other AWS regions#39045
tigrato merged 3 commits intomasterfrom
tigrato/aws-missing-certs

Conversation

@tigrato
Copy link
Copy Markdown
Contributor

@tigrato tigrato commented Mar 6, 2024

This PR extends the support for EC2 joining method for the following AWS regions:

  • ap-south-2
  • ap-southeast-3
  • ap-southeast-4
  • ca-west-1
  • eu-central-2
  • eu-south-2
  • il-central-1
  • me-central-1

Changelog: Expand the EC2 joining process to include newly created AWS regions

This PR extends the support for EC2 joining method for the following AWS
regions:

- ap-south-2
- p-southeast-3
- ap-southeast-4
- ca-west-1
- eu-central-2
- eu-south-2
- il-central-1
- me-central-1

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
Copy link
Copy Markdown
Contributor

@reedloden reedloden left a comment

Choose a reason for hiding this comment

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

Perhaps sort this alphabetically to make it easier to see if a region is missing?

@tigrato
Copy link
Copy Markdown
Contributor Author

tigrato commented Mar 6, 2024

Perhaps sort this alphabetically to make it easier to see if a region is missing?

done in d65cc04

Comment thread lib/auth/aws_certs.go Outdated
@tigrato tigrato requested a review from reedloden March 6, 2024 23:38
Copy link
Copy Markdown
Contributor

@reedloden reedloden left a comment

Choose a reason for hiding this comment

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

Spot checked each one against https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/regions-certs.html (RSA 2048 tab) and confirmed they look correct. Thanks!

@tigrato tigrato added this pull request to the merge queue Mar 7, 2024
Merged via the queue into master with commit ae847a7 Mar 7, 2024
@tigrato tigrato deleted the tigrato/aws-missing-certs branch March 7, 2024 09:58
@public-teleport-github-review-bot
Copy link
Copy Markdown

@tigrato See the table below for backport results.

Branch Result
branch/v13 Create PR
branch/v14 Create PR
branch/v15 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants