tflint-ruleset-google

tflint-ruleset-google copied to clipboard

Better support and/or documentation for Credentials related to Deep Checking

1

From the [Deep Checking](https://github.com/terraform-linters/tflint-ruleset-google/blob/master/docs/deep_checking.md) documentation it says: > Currently, credentials, regions, etc. declared inside the "google" provider block are not considered except for the project attribute. You need to pass...

nshenry03

[proposal] Warn google_access_context_manager_service_perimeter_resource without ignore_changes

1

See https://www.terraform.io/docs/providers/google/r/access_context_manager_service_perimeter_resource.html > Note: If this resource is used alongside a google_access_context_manager_service_perimeter resource, the service perimeter resource must have a lifecycle block with ignore_changes = [status[0].resources] so they don't fight...

wata727

[proposal] Warn google_binary_authorization_attestor_iam for conflicting

3

See https://www.terraform.io/docs/providers/google/r/binary_authorization_attestor_iam.html > Note: google_binary_authorization_attestor_iam_policy cannot be used in conjunction with google_binary_authorization_attestor_iam_binding and google_binary_authorization_attestor_iam_member or they will fight over what your policy should be. > Note: google_binary_authorization_attestor_iam_binding resources can be...

wata727

Version 0.18.0 and 0.15.0 failures on Ubuntu 22.04. Version 0.17.0 works

1

I'm working with a fresh install of Ubuntu 22.04, using `go1.18.1 linux/amd64`, `TFLint version 0.38.1`, and `pre-commit 2.19.0`. Our team uses tflint with the tflint-ruleset-google plugin on a daily basis,...

donjmilligan

copied from https://github.com/terraform-linters/tflint-ruleset-google/pull/174 similar to: https://github.com/terraform-linters/tflint-ruleset-aws/pull/290, https://github.com/terraform-linters/tflint-ruleset-azurerm/pull/172 currently running and updating this PR in my fork: https://github.com/PatMyron/tflint-ruleset-google/pull/1 **not sure why it imported `fmt`/`regexp` everywhere and not sure I understand the...

PatMyron

[proposal] Warn google_bigtable_instance without prevent_destroy

3

See https://www.terraform.io/docs/providers/google/r/bigtable_instance.html > Note: It is strongly recommended to set lifecycle { prevent_destroy = true } on instances in order to prevent accidental data loss. See Terraform docs for more...

wata727

[proposal] Warn google_compute_instance_group without create_before_destroy

1

See https://www.terraform.io/docs/providers/google/r/compute_instance_group.html > Recreating an instance group that's in use by another resource will give a resourceInUseByAnotherResource error. You can avoid this error with a Terraform lifecycle block as outlined...

wata727

See https://www.terraform.io/docs/providers/google/r/container_cluster.html > The Google provider is unable to validate certain configurations of private_cluster_config when enable_private_nodes is false. It's recommended that you omit the block entirely if the field is...

wata727

See https://www.terraform.io/docs/providers/google/r/compute_region_disk_resource_policy_attachment.html > Note: This resource does not support zonal disks (google_compute_disk). For zonal disks, please refer to google_compute_disk_resource_policy_attachment

wata727

See https://www.terraform.io/docs/providers/google/r/compute_disk_resource_policy_attachment.html > Note: This resource does not support regional disks (google_compute_region_disk). For regional disks, please refer to google_compute_region_disk_resource_policy_attachment

wata727