Skip to content

[v18] Add tsh aws-profile to Generate AWS Profiles from AWS Identity Center Integration#64590

Draft
tele-lion wants to merge 2 commits intobranch/v18from
tele-lion/backport-63032-64397-branch/v18
Draft

[v18] Add tsh aws-profile to Generate AWS Profiles from AWS Identity Center Integration#64590
tele-lion wants to merge 2 commits intobranch/v18from
tele-lion/backport-63032-64397-branch/v18

Conversation

@tele-lion
Copy link
Contributor

@tele-lion tele-lion commented Mar 12, 2026

Backport of the following PRs to branch/v18:

Manual Test Plan

Test Environment

Test Cases

  • test

changelog: Added a new tsh aws-profile command that detects your AWS Identity Center integration (if configured) and writes corresponding AWS profiles into your local AWS config file for later use

* created two AWS IC constants for account and sso region

* fix inconsistency
@github-actions github-actions bot added backport size/lg tsh tsh - Teleport's command line tool for logging into nodes running Teleport. labels Mar 12, 2026
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: aa51b0cb30

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".


// Prepare AWS profile for the combination of each permission set and account.
for _, ps := range awsIC.PermissionSets {
profileName := formatAWSProfileName(accountName, ps.Name)

Choose a reason for hiding this comment

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

P1 Badge Make generated AWS profile names globally unique

The profile name is derived only from accountName and permission set (formatAWSProfileName(accountName, ps.Name)), so two Identity Center accounts from different sessions/integrations that share the same account label and role (for example, both have dev + Admin) produce the same [profile ...] section name. WriteSSOConfig then reuses that section and the later app silently overwrites sso_session/sso_account_id, leaving only one working profile and potentially pointing users at the wrong account.

Useful? React with 👍 / 👎.

@tele-lion tele-lion marked this pull request as draft March 12, 2026 23:07
@tele-lion tele-lion changed the title [v18] Backport PRs 63032-64397 [v18] Add tsh aws-profile to Generate AWS Profiles from AWS Identity Center Integration Mar 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport size/lg tsh tsh - Teleport's command line tool for logging into nodes running Teleport.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant