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

fix: Allow customisation of trusted_role_actions in iam-assumable-role module #76

Conversation

victormesas
Copy link
Contributor

Description

Add var to action to can fill out in module iam-assumable-role

Motivation and Context

We use github actions and the AWS action to get credentials need the sts:TagSession permission as you can read on:
https://github.com/aws-actions/configure-aws-credentials/

Breaking Changes

Doesn't apply

How Has This Been Tested?

I tested this changes on my AWS account to ensure that works as I spected

@victormesas victormesas changed the title Feature/var trusted role actions fix: var trusted role actions May 20, 2020
@victormesas
Copy link
Contributor Author

Hi @antonbabenko can you take a look to this PR?

Thanks in advice

@antonbabenko antonbabenko changed the title fix: var trusted role actions fix: Allow customisation of trusted_role_actions in iam-assumable-role module May 26, 2020
@antonbabenko antonbabenko merged commit 5bb2ab9 into terraform-aws-modules:master May 26, 2020
@antonbabenko
Copy link
Member

Thanks @victormesas for this PR and for sharing a link to the repo.

v2.10.0 has been just released.

@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 10, 2022
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.

2 participants