Skip to content

feat: add node IAM additional policies variable#98

Open
kav wants to merge 1 commit intomainfrom
feat/node-iam-policies
Open

feat: add node IAM additional policies variable#98
kav wants to merge 1 commit intomainfrom
feat/node-iam-policies

Conversation

@kav
Copy link
Member

@kav kav commented Feb 26, 2026

Summary

  • Adds node_iam_additional_policies variable (map(string), default {})
  • Wires it into the default EKS managed node group as iam_role_additional_policies
  • Allows attaching extra IAM policies to node group roles (e.g., KMS decrypt, SSM, custom policies)

Test plan

  • Run tofu validate to verify syntax
  • Run tofu plan with default (empty map) — no changes expected
  • Run tofu plan with node_iam_additional_policies = { ssm = "arn:aws:iam::aws:policy/AmazonSSMManagedInstanceCore" } — policy attachment should appear

🤖 Generated with Claude Code

@kav kav force-pushed the feat/node-iam-policies branch from 54c83d8 to 6b4efc5 Compare February 26, 2026 23:22
@kav kav force-pushed the feat/node-iam-policies branch from 6b4efc5 to 390b24c Compare February 26, 2026 23:25
@josmo
Copy link
Member

josmo commented Mar 5, 2026

this seems to have a bad merge? @kav are you letting claude run the show? ;)

@kav
Copy link
Member Author

kav commented Mar 5, 2026

Oh man, probably not embarassingly enough, the number of merge conflicts in this set was nuts

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.

3 participants