Skip to content

fix: Work around path bug in aws-iam-authenticator#894

Merged
dpiddockcmp merged 3 commits into
terraform-aws-modules:masterfrom
DWSR:support-iam-path
May 30, 2020
Merged

fix: Work around path bug in aws-iam-authenticator#894
dpiddockcmp merged 3 commits into
terraform-aws-modules:masterfrom
DWSR:support-iam-path

Conversation

@DWSR

@DWSR DWSR commented May 27, 2020

Copy link
Copy Markdown
Contributor

PR o'clock

Description

aws-iam-authenticator has an open issue where it will not recognize
IAM roles that include paths. This change causes the path supplied to
var.iam_path to be stripped when generating the aws-auth ConfigMap
in order to work around this.

kubernetes-sigs/aws-iam-authenticator#153

Fixes #893

Checklist

Brandon McNama added 2 commits May 26, 2020 23:52
`aws-iam-authenticator` has an open issue where it will not recognize
IAM roles that include paths. This change causes the path supplied to
`var.iam_path` to be stripped when generating the `aws-auth` ConfigMap
in order to work around this.

kubernetes-sigs/aws-iam-authenticator#153
@DWSR DWSR marked this pull request as ready for review May 27, 2020 04:08
@dpiddockcmp dpiddockcmp merged commit 7ffe5fa into terraform-aws-modules:master May 30, 2020
@dpiddockcmp

Copy link
Copy Markdown
Contributor

Thank you for the fix

@github-actions

Copy link
Copy Markdown

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 18, 2022
@DWSR DWSR deleted the support-iam-path branch November 18, 2022 05:23
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.

Specifying iam_path causes nodes to be unable to join cluster

2 participants