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 example of AWS AssumeRole #165

Merged
merged 2 commits into from
Nov 6, 2018
Merged

Add example of AWS AssumeRole #165

merged 2 commits into from
Nov 6, 2018

Conversation

jen20
Copy link
Contributor

@jen20 jen20 commented Nov 6, 2018

This pull request adds an example of using AssumeRole with the AWS provider. It is split into two parts, one which creates an unprivileged user and a role for it to assume, and one which creates a resource under the security context of the assumed role.

This seems to have been a common request from the community - the specific motivating question for this is here.

Copy link
Member

@pgavlin pgavlin left a comment

Choose a reason for hiding this comment

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

LGTM modulo a couple nits

aws-ts-assume-role/README.md Outdated Show resolved Hide resolved
Copy link
Member

@joeduffy joeduffy left a comment

Choose a reason for hiding this comment

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

Love it!

@jen20 jen20 force-pushed the assume-role-demo branch 2 times, most recently from 9a5c960 to 8699453 Compare November 6, 2018 02:51
@jen20 jen20 merged commit 85bcce5 into master Nov 6, 2018
@jen20 jen20 deleted the assume-role-demo branch November 6, 2018 02:53
ramene pushed a commit to ramene/pulumi-kubeflow-ml that referenced this pull request Sep 7, 2019
ramene pushed a commit to ramene/pulumi-kubeflow-ml that referenced this pull request Sep 13, 2019
dixler pushed a commit that referenced this pull request Jan 21, 2022
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.

None yet

4 participants