Nick Elliot
Nick Elliot
Followed the testing steps mentioned and the testing image is referenced in the execution details. Believe it would involve undoing the "testing" reference before this is merged, and pushing up...
cc @c2thorn
Looking into this issue as on-call, currently two comments: 1. The base module configuration is bad, [the setup of](https://github.com/GoogleCloudPlatform/terraform-google-alloy-db/blob/4037cda46d5d51595cdda5c2c5e18d2e80fcf27b/main.tf#L44): ``` network_config { network = var.network_self_link allocated_ip_range = var.allocated_ip_range } dynamic...
Yeah thats why I mentioned "This doesn't change the fact the issue persists even in configs where this part is corrected, but the module likely does need to change in...
@SarahFrench that sounds good to me, feel free to merge this once those have been done!
To explain part of this as the one who implemented the change: the resources without the defaultprovider/region/zone displays were those that do not support using system default values on the...
/gcbrun
/gcbrun
oops, didnt mean to convert out of draft, sorry!
Implemented via modified `getProviderDefaultFromFrameworkSchema` function