-
Notifications
You must be signed in to change notification settings - Fork 0
PLT-1253: Remove legacy terraform backends #283
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR should update backends to drop the -gf
suffix and update workflows to point at non-gf backends.
…dc-provider │ does not exist
…dc-provider │ does not exist
…dc-provider │ does not exist
83f6c18
to
4242d1d
Compare
|
All checks now passing. got a .terraform/modules/iam_github_oidc_provider/modules/iam-github-oi… |
Looks like that rename was merged a couple of days ago terraform-aws-modules/terraform-aws-iam#585 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like the api_customers IP set for AB2D is now handled in their app repo, so modify the api-waf service and remove it from that state.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm seeing other changes to the WAF rules in the api-waf plan on AB2D environments. Let's remove ab2d from the tf-api-waf workflow matrix and undo the changes made to the api-waf service.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The file at terraform/services/github-actions-oidc-provider/.terraform.lock.hcl should be removed from version control.
.terraform.lock.hcl from version control has been dropped |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Thanks for working in changes
🎫 Ticket
https://jira.cms.gov/browse/PLT-1253
🛠 Changes
legacy terraform backends were removed
ℹ️ Context
With legacy accounts deleted, the is the need to remove terraform backends for this legacy accounts
🧪 Validation
See checks.