Scott Suarez

Results 123 comments of Scott Suarez

This appears to be a breaking change, see https://github.com/GoogleCloudPlatform/magic-modules/pull/14300#issuecomment-2984189491

> > This appears to be a breaking change, see [#14300 (comment)](https://github.com/GoogleCloudPlatform/magic-modules/pull/14300#issuecomment-2984189491) > > yes it's a breaking change, so created an PR on top 7.0 feature branch, Anything else...

Looks like merging main into this branch has added unintended changes here.

> > If the field is currently Computed, do we know why it was marked as such? i.e Was there any change from the API side which mandated this field...

I've looked into solving this but there isn't a simple fix at the generator level without introducing complexity or new implied behavior. I can think for 5 potential solutions to...

Still seeing failures here https://hashicorp.teamcity.com/test/-224300369035462042?currentProjectId=TerraformProviders_GoogleCloud_GOOGLE_BETA_NIGHTLYTESTS&expandTestHistoryChartSection=true ``` === RUN TestAccSecurityCenterManagementOrganizationEventThreatDetectionCustomModule === PAUSE TestAccSecurityCenterManagementOrganizationEventThreatDetectionCustomModule === CONT TestAccSecurityCenterManagementOrganizationEventThreatDetectionCustomModule vcr_utils.go:152: Step 1/4 error: Error running apply: exit status 1 Error: Error creating OrganizationEventThreatDetectionCustomModule: googleapi:...

Actually.. I'm going to continue to keep a pulse on this. Since the change has only been checked in recently I'll let this play out a bit more.

Looks like it's still crashing downstream https://github.com/GoogleCloudPlatform/magic-modules/actions/runs/11464698109/job/31901478012?pr=12013