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

feat: Enabling support for Pod Identity #30

Closed
wants to merge 5 commits into from
Closed

Conversation

rodrigobersa
Copy link
Collaborator

What does this PR do?

  • Adding Pod Identity Association Resource
  • Updating IAM Policy to provide required Pod Identity trust policy
  • Adding variables to support Pod Identity
  • Adding outputs to consume Pod Identity configuration

Motivation

Pod Identity aims to be the default option for granting permissions to addons. We should adopt this in the v2.

Test Results

Addons PR

Additional Notes

$ pre-commit run --files *
Terraform fmt............................................................Passed
Terraform validate.......................................................Passed
Terraform docs...........................................................Passed
Terraform validate with tflint...........................................Passed
check for merge conflicts................................................Passed
fix end of files.........................................................Passed
```

@rodrigobersa rodrigobersa requested a review from a team as a code owner June 9, 2024 23:25
Copy link

This PR has been automatically marked as stale because it has been open 30 days
with no activity. Remove stale label or comment or this PR will be closed in 10 days

@github-actions github-actions bot added the stale label Jul 10, 2024
Copy link

Pull request closed due to inactivity.

@github-actions github-actions bot closed this Jul 20, 2024
@rodrigobersa rodrigobersa reopened this Jul 30, 2024
@rodrigobersa rodrigobersa marked this pull request as draft July 30, 2024 16:15
@github-actions github-actions bot removed the stale label Jul 31, 2024
@rodrigobersa rodrigobersa marked this pull request as ready for review August 5, 2024 21:37
@tbulding tbulding marked this pull request as draft August 5, 2024 22:19
@tbulding tbulding marked this pull request as ready for review August 5, 2024 22:19
@tbulding tbulding marked this pull request as draft August 5, 2024 22:59
@tbulding tbulding marked this pull request as ready for review August 5, 2024 22:59
@tbulding tbulding marked this pull request as draft August 5, 2024 23:00
@tbulding tbulding marked this pull request as ready for review August 5, 2024 23:00
@rodrigobersa rodrigobersa reopened this Aug 5, 2024
@rodrigobersa rodrigobersa reopened this Aug 5, 2024
@rodrigobersa rodrigobersa reopened this Aug 5, 2024
@rodrigobersa rodrigobersa reopened this Aug 5, 2024
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.

1 participant