Stephen Lewis (Burrows)
Stephen Lewis (Burrows)
manual run of TestAccSqlDatabaseInstance_mysqlMajorVersionUpgrade: https://ci-oss.hashicorp.engineering/buildConfiguration/GoogleCloudBeta_ProviderGoogleCloudBetaMmUpstream/327532
@SarahFrench in terms of the fix here, what's the recommended path forward? It looks like the fields in question are both optional + computed, which is why they persist values.
Thanks! I wonder if it might also be possible to use encoders / decoders to handle this better, similar to what's described in https://github.com/hashicorp/terraform-provider-google/issues/14903#issuecomment-1764923141, since it's a complex API-side default....
No longer seems to be failing this way.
go/terraform-on-cgc may be useful if desired.
Some work remaining on this.
This was resolved by https://github.com/GoogleCloudPlatform/magic-modules/pull/6617
I'm going to change this back to a bug since it's an issue with the implementation rather than a coverage gap, and it seems like the current behavior is problematic.