Use Pod Identity instead of IRSA#3171
Closed
Adam-D-Lewis wants to merge 3 commits intoclaude/investigate-issue-011CUQHTM3P2xpX98sxMQ2aXfrom
Closed
Use Pod Identity instead of IRSA#3171Adam-D-Lewis wants to merge 3 commits intoclaude/investigate-issue-011CUQHTM3P2xpX98sxMQ2aXfrom
Adam-D-Lewis wants to merge 3 commits intoclaude/investigate-issue-011CUQHTM3P2xpX98sxMQ2aXfrom
Conversation
This commit migrates the Cluster Autoscaler from IRSA (IAM Roles for Service Accounts) to the newer EKS Pod Identity feature. Changes: - Add eks-pod-identity-agent addon to EKS cluster - Replace OIDC-based trust policy with Pod Identity trust policy - Add aws_eks_pod_identity_association resource to link IAM role to service account - Remove eks.amazonaws.com/role-arn annotation from Helm service account - Simplify configuration by removing complex OIDC conditions Benefits of Pod Identity over IRSA: - Simpler setup without OIDC provider complexity - Better scalability (no IAM trust policy size limits) - Easier multi-cluster management (same role works across clusters) - AWS's recommended approach going forward 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This commit completes the migration to Pod Identity by also converting the EBS CSI Driver from IRSA to Pod Identity. Changes: - Replace OIDC-based trust policy with Pod Identity trust policy for EBS CSI Driver - Add aws_eks_pod_identity_association for EBS CSI Driver - Remove complex OIDC conditions Now both the Cluster Autoscaler and EBS CSI Driver use Pod Identity instead of IRSA, providing a consistent and simplified IAM configuration. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This commit removes all IRSA (IAM Roles for Service Accounts) code now that we've fully migrated to Pod Identity. Changes: - Remove OIDC provider and tls_certificate data source - Remove cluster_oidc_issuer_url and oidc_provider_arn outputs - Remove cluster_autoscaler_role_arn output and variable passing - Remove iam_role_arn variable from cluster-autoscaler module - Clean up InputVars class in kubernetes_initialize stage The codebase now exclusively uses EKS Pod Identity for both: - Cluster Autoscaler - EBS CSI Driver 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
11 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Converts IRSA to Pod Identity