-
-
Notifications
You must be signed in to change notification settings - Fork 945
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
docs: Updated example in README #52
Conversation
@antonbabenko @max-rocket-internet |
@max-rocket-internet Can it be merged? |
Hi Arie,
I will merge on Monday.
… On 29 Feb 2020, at 20:08, Arie Lev ***@***.***> wrote:
@max-rocket-internet Can it be merged?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
@antonbabenko |
* upstream/master: (26 commits) Updated CHANGELOG fix: Fixed ses_smtp_password_v4 output name Updated CHANGELOG fix: simplify count statements (terraform-aws-modules#93) Updated CHANGELOG fix: Allow running on custom AWS partition (incl. govcloud) (terraform-aws-modules#94) Updated CHANGELOG feat: modules/iam-assumable-role-with-oidc: Support multiple provider URLs (terraform-aws-modules#91) Updated CHANGELOG feat: Strip https:// from OIDC provider URL if present (terraform-aws-modules#50) Updated CHANGELOG fix: Allow modules/iam-assumable-role-with-oidc to work in govcloud (terraform-aws-modules#83) Updated CHANGELOG feat: Added support for sts:ExternalId in modules/iam-assumable-role (terraform-aws-modules#90) Updated CHANGELOG fix: Delete DEPRECATED ses_smtp_password in iam-user. (terraform-aws-modules#88) Updated CHANGELOG feat: Support for Terraform v0.13 and AWS provider v3 (terraform-aws-modules#87) docs: Updated example in README (terraform-aws-modules#52) Updated CHANGELOG ...
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. |
Description
While trying to implement the
iam-assumable-role-with-oidc
module, it didn't worked out of the box,then i noticed the readme used a wrong module the
iam-assumable-role
.