Stephen Lewis (Burrows)

@ramonvermeulen After talking things over with @BBBmau, we'd actually like to ask you to make the generator changes _first_, then the resource changes after. We're working on putting together a...

@BBBmau wouldn't it be easier to merge each part of the fix directly into main like normal? The incremental changes should be safe?

we should avoid doing a single large branch. That said, each incremental change will need to be _complete_. For example, generating expander behavior should be in a separate PR from...

regarding the UX, see https://github.com/hashicorp/terraform-provider-google/issues/23213#issuecomment-2959923206

Adding the new fields should happen at yaml parsing time, so that they look like they've "always been there" and can be handled like any other fields. It should be...

@zli82016 possibly this is expected? The diff looks like: ``` resource "google_cloud_run_service" "default" { id = "locations/us-central1/namespaces/ci-test-project-188019/services/tftest-cloudrun-lohq0k" name = "tftest-cloudrun-lohq0k" # (4 unchanged attributes hidden) ~ metadata { ~ effective_annotations...

I recently ran into this as well - specifically the part where suppressing a diff on the `#` field causes the entire value to be seen as empty when applying...

Hi! It sounds like you're using a module, which make this hard to diagnose. Could you provide a small reproducible Terraform file that can reproduce the issue?

Due to the number of tests impacted, we probably can't run them all in serial without making test runs take unacceptably long. We'd need to figure out a way to...

Note from triage: This may not make sense in Terraform, since Terraform is a declarative tool, but forwarding to engineering to make that determination. If you have any additional thoughts...