Skip to content

Conversation

@melinath
Copy link
Member

@melinath melinath commented Oct 30, 2025

Reverts #15489

That PR introduced some custom code that's not behaving properly because it uses resource.Retry (which comes from a testing package) instead of transport_tpg.Retry as expected.

`google_saas_runtime_unit_operation` (revert)

@roaks3 roaks3 self-requested a review October 30, 2025 21:04
@modular-magician
Copy link
Collaborator

Hi there, I'm the Modular magician. I've detected the following information about your changes:

Diff report

Your PR generated some diffs in downstreams - here they are.

google provider: Diff ( 1 file changed, 477 deletions(-))
google-beta provider: Diff ( 6 files changed, 2 insertions(+), 2178 deletions(-))
terraform-google-conversion: Diff ( 1 file changed, 309 deletions(-))

Breaking Change(s) Detected

The following breaking change(s) were detected within your pull request.

  • Resource google_saas_runtime_unit_operation was either removed or renamed - reference

If you believe this detection to be incorrect please raise the concern with your reviewer.
If you intend to make this change you will need to wait for a major release window.
An override-breaking-change label can be added to allow merging.

Errors

google-beta provider:

  • Error reading file website/docs/r/saas_runtime_unit_operation.html.markdown

@melinath melinath added this pull request to the merge queue Oct 30, 2025
@modular-magician
Copy link
Collaborator

Tests analytics

Total tests: 12
Passed tests: 12
Skipped tests: 0
Affected tests: 0

Click here to see the affected service packages
  • saasruntime

🟢 All tests passed!

View the build log

Merged via the queue into main with commit 4d1cb60 Oct 30, 2025
29 of 31 checks passed
@melinath melinath deleted the revert-15489-saasruntime-autogen-unitoperation branch October 30, 2025 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants