Skip to content

Adjust role split string to not hardcode 'aws' partition of AWS#2042

Merged
felixfontein merged 1 commit intogetsops:mainfrom
felixfontein:arn
Feb 16, 2026
Merged

Adjust role split string to not hardcode 'aws' partition of AWS#2042
felixfontein merged 1 commit intogetsops:mainfrom
felixfontein:arn

Conversation

@felixfontein
Copy link
Copy Markdown
Contributor

This also makes it possible to use AWS's new ECS (European Sovereign Cloud), which uses the partition aws-eusc according to https://aws.amazon.com/blogs/aws/opening-the-aws-european-sovereign-cloud/.

Reference on ARNs: https://docs.aws.amazon.com/IAM/latest/UserGuide/reference-arns.html

Fixes #805.
Closes #806.

@felixfontein
Copy link
Copy Markdown
Contributor Author

Ping @getsops/maintainers.

Copy link
Copy Markdown
Contributor

@sabre1041 sabre1041 left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Felix Fontein <felix@fontein.de>
@felixfontein felixfontein merged commit 7153e40 into getsops:main Feb 16, 2026
15 checks passed
@felixfontein felixfontein deleted the arn branch February 16, 2026 06:09
@felixfontein
Copy link
Copy Markdown
Contributor Author

@sabre1041 thanks for reviewing this!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

kms+role doesnt work properly with aws govcloud partition

2 participants