Skip to content
This repository was archived by the owner on Feb 23, 2026. It is now read-only.

fix: add actionable errors for GCE long running operations - #498

Merged
parthea merged 8 commits into
mainfrom
add-actionable-errors-lro
Jun 12, 2023
Merged

fix: add actionable errors for GCE long running operations#498
parthea merged 8 commits into
mainfrom
add-actionable-errors-lro

Conversation

@parthea

@partheaparthea commented May 19, 2023

Copy link
Copy Markdown
Contributor

Fixes b/280938007

Googlers see go/surface-actionable-errors-lro-gce

When an error occurs during an extended operation when using the python compute client, currently users only receive the error code and the error message, which is not helpful.

google.api_core.exceptions.BadRequest: 400 BAD REQUEST

The proposed solution is to provide users with actionable error information, surfacing the errors field which contains actionable errors.

google.api_core.exceptions.BadRequest: 400 BAD REQUEST
INCOMPATIBLE_BACKEND_SERVICES: Validation failed for instance group 'projects/<REDACTED>/zones/us-central1-a/instanceGroups/my-instance-group-1': backend services 'projects/<REDACTED>/global/backendServices/my-backend-service-1' and 'projects/<REDACTED>global/backendServices/my-backend-service-2' point to the same instance group but the backends have incompatible balancing_mode. Values should be the same.

@product-auto-labelproduct-auto-labelBot added the size: s Pull request size is small. label May 19, 2023
@parthea
partheaforce-pushed the add-actionable-errors-lro branch 2 times, most recently from 50b5a79 to 254e151CompareMay 19, 2023 17:04
@parthea
partheaforce-pushed the add-actionable-errors-lro branch from 254e151 to 0e3f9bdCompareMay 19, 2023 18:44

@partheaparthea left a comment

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll add tests if go/surface-actionable-errors-lro-gce is approved.

@product-auto-labelproduct-auto-labelBot added size: m Pull request size is medium. and removed size: s Pull request size is small. labels Jun 1, 2023
@parthea
parthea marked this pull request as ready for review June 1, 2023 14:34
@parthea
parthea requested review from a teamJune 1, 2023 14:34
@parthea
parthea enabled auto-merge (squash) June 12, 2023 15:58
@parthea
parthea merged commit 7dfc3a7 into mainJun 12, 2023
@parthea
parthea deleted the add-actionable-errors-lro branch June 12, 2023 16:00
@release-pleaserelease-pleaseBot mentioned this pull request Jun 12, 2023
This was referenced May 30, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size: mPull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@parthea@vchudnov-g@dizcology