Update TF plan/apply from PR 146 to use separate OIDC roles#142
Conversation
|
Terraform plan in terraform No changes. Your infrastructure matches the configuration.✅ Plan applied in Terraform apply (OIDC) #58 |
|
This can be applied once the issue with reading secrets in devops-security is fixed - @Benettonkkb please complete hackforla/devops-security#154 when you can |
There was a problem hiding this comment.
Pull request overview
Updates GitHub Actions Terraform workflows to assume separate AWS OIDC roles for plan vs apply, aligning CI permissions with least-privilege separation goals. The PR also modifies the shared CI/CD IAM policy used by project pipelines.
Changes:
- Update Terraform plan workflow to assume
incubator-tf-plan. - Update Terraform apply workflow to assume
incubator-tf-apply. - Adjust
terraform/cicd.tfIAM policy formatting and add Secrets Manager read access forhome-unite-us-*.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| terraform/cicd.tf | Reformat IAM policy statements and add Secrets Manager GetSecretValue permission for home-unite-us-*. |
| .github/workflows/terraform-plan.yaml | Switch Terraform plan job to assume the incubator-tf-plan role. |
| .github/workflows/terraform-apply.yaml | Switch Terraform apply job to assume the incubator-tf-apply role. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
28f7d8c to
592ec23
Compare
|
used |
Part 2 of Pull request 147
What changes did you make?
incubator-tf-planandincubator-tf-applytherole-to-assumein the...plan.tfandapply.tffilesWhy did you make the changes (we will use this info to test)?