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: change IAM role policy to allow dynamic volume provisioning #34

Merged
merged 2 commits into from
Aug 7, 2024

Conversation

lconsuegra
Copy link
Member

Description of the changes

I tried to do dynamic provisioning with this module but it does not work because of permissions. There is a AWS managed policy for the efs driver so I propose we use it.

The current actions are embedded in the AmazonEFSCSIDriverPolicy.

Breaking change

  • No

Tests executed on which distribution(s)

  • EKS (AWS)

@lconsuegra lconsuegra self-assigned this Aug 2, 2024
@lconsuegra lconsuegra requested a review from a team as a code owner August 2, 2024 09:45
@lentidas lentidas self-requested a review August 7, 2024 12:23
@lentidas lentidas removed their assignment Aug 7, 2024
@lentidas lentidas changed the title chore: change IAM role policy fix: change IAM role policy to allow dynamic volume provisioning Aug 7, 2024
@lentidas lentidas merged commit 2d258fb into main Aug 7, 2024
@lentidas lentidas deleted the chore/change_iam_policy branch August 7, 2024 12:25
@github-actions github-actions bot mentioned this pull request Aug 7, 2024
@lentidas
Copy link
Contributor

lentidas commented Aug 7, 2024

@lconsuegra I've changed the PR title to fix in order to trigger a release and went ahead and merged the PR ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants