Skip to content
This repository was archived by the owner on Nov 5, 2024. It is now read-only.

AWS sources and targets cross-account/remote IAM support#1400

Merged
tzununbekov merged 2 commits into
mainfrom
cross-account-iam
May 3, 2023
Merged

AWS sources and targets cross-account/remote IAM support#1400
tzununbekov merged 2 commits into
mainfrom
cross-account-iam

Conversation

@tzununbekov

@tzununbekov tzununbekov commented Apr 27, 2023

Copy link
Copy Markdown
Member

AWS components' auth support for cross-account IAM impersonation.

Available auth schemas:

static creds

  ...
  auth:
    credentials:
      accessKeyID:
        value: <redacted>
      secretAccessKey:
        value: <redacted>

-client session is created with the IAM user account whose Access Key specified in the spec. Controller also uses the same access key.

IAM Role

  ...
  auth:
    iamRole: <redacted>

-IAM role impersonation available on EKS only. New SA will be created for the adapter, controller's SA requires manual update to include EKS role annotation. STS trust relationship configuration required on IAM.

New: cross-account IAM Role

  ...
  auth:
    credentials:
      accessKeyID:
        value: <redacted>
      secretAccessKey:
        value: <redacted>
      assumeIamRole: <redacted>

-after client session is created with the static credentials, assumeIamRole will be used to gain IAM permissions. STS trust relationship configuration required on IAM.

Resolves #1253

@tzununbekov tzununbekov self-assigned this Apr 27, 2023
@tzununbekov tzununbekov force-pushed the cross-account-iam branch from 040976a to e33b809 Compare May 1, 2023 05:05
@tzununbekov tzununbekov force-pushed the cross-account-iam branch from e33b809 to 7c65d59 Compare May 1, 2023 05:27
@tzununbekov tzununbekov force-pushed the cross-account-iam branch from 229b44f to 84f148a Compare May 1, 2023 07:43
@tzununbekov tzununbekov requested a review from a team May 1, 2023 12:00
@tzununbekov tzununbekov marked this pull request as ready for review May 1, 2023 12:01
@tzununbekov

Copy link
Copy Markdown
Member Author

@FranBarrera thanks!
@jmcx this update may need to be reflected in the docs. Not breaking nor critical though.

@tzununbekov tzununbekov merged commit 79f51b4 into main May 3, 2023
@tzununbekov tzununbekov deleted the cross-account-iam branch May 3, 2023 10:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support for assume role

2 participants