Closed brettcurtis closed 1 month ago
Consider fixing these issues, they don't align with your company's FinOps policies & the Well-Architected Framework.
Tagging policies | |
β Default | |
resource [google_compute_global_address.istio_gateway[0]](https://github.com/osinfra-io/terraform-kubernetes-istio/blob/HEAD/regional/main.tf#L73) provisioned by module [module.kubernetes_istio](https://github.com/osinfra-io/google-cloud-kubernetes/blob/af02ffb68b056c82ca202e3dbe2b7c105e9d361c/regional/istio/main.tf#L73) * Missing mandatory tags: ` team `, ` cost-center `, ` env `, ` repository `. Consider using default tags to avoid adding tags to individual resources. in projects ` regional-istio-production `, ` regional-istio-sandbox ` | |
resource [google_compute_global_address.istio_gateway_mci[0]](https://github.com/osinfra-io/terraform-kubernetes-istio/blob/HEAD/main.tf#L92) provisioned by module [module.kubernetes_istio](https://github.com/osinfra-io/google-cloud-kubernetes/blob/af02ffb68b056c82ca202e3dbe2b7c105e9d361c/main.tf#L92) * Missing mandatory tags: ` team `, ` cost-center `, ` env `, ` repository `. Consider using default tags to avoid adding tags to individual resources. in projects ` main-production `, ` main-sandbox ` |
This comment will be updated when code changes.
The recent modifications represent a significant renaming effort throughout the configuration files, primarily transitioning identifiers from "global" to "main." This strategic shift enhances clarity and consistency within the Terraform and GitHub Actions workflows, effectively aligning naming conventions with their intended purpose. The updates encompass job identifiers, data source references, and variable declarations, thereby fostering a more organized and intuitive infrastructure management structure.
Files | Change Summary |
---|---|
.github/workflows/sandbox.yml |
Renamed job from global to main , updated parameters github_environment and terraform_workspace accordingly. |
.pre-commit-config.yaml |
Updated versions of pre-commit-terraform and checkov to newer releases. |
regional/README.md , regional/mci/README.md , regional/onboarding/README.md |
Changed references from terraform_remote_state.global to terraform_remote_state.main for consistency in documentation. |
regional/locals.tf , regional/mci/locals.tf |
Renamed local variable from global to main , updating references to their respective outputs. |
regional/main.tf , regional/mci/main.tf , regional/onboarding/main.tf |
Updated terraform_remote_state data source and associated workspace naming from global to main , reflecting new conventions. |
regional/outputs.tf |
Changed project_id output source from local.global.project_id to local.main.project_id . |
shared/tfvars/non-production.tfvars , shared/tfvars/production.tfvars , shared/tfvars/sandbox.tfvars |
Removed namespace configurations for backstage , istio-ingress , and istio-system , indicating a shift in namespace management. |
Summary by CodeRabbit
New Features
Documentation
Chores
Bug Fixes