FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix: add actionable errors for GCE long running operations by parthea · Pull Request #498 · googleapis/python-api-core · GitHub

This repository was archived by the owner on Feb 23, 2026. It is now read-only.
/ python-api-core Public archive

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 commented May 19, 2023
edited
Loading

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-label Bot added the size: s Pull request size is small. label May 19, 2023
parthea force-pushed the add-actionable-errors-lro branch 2 times, most recently from 50b5a79 to 254e151 Compare May 19, 2023 17:04
parthea force-pushed the add-actionable-errors-lro branch from 254e151 to 0e3f9bd Compare May 19, 2023 18:44

parthea left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

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

product-auto-label Bot added size: m Pull request size is medium. and removed size: s Pull request size is small. labels Jun 1, 2023
parthea marked this pull request as ready for review June 1, 2023 14:34
parthea requested review from a team June 1, 2023 14:34
parthea requested review from dizcology and vchudnov-g June 1, 2023 14:41
parthea enabled auto-merge (squash) June 12, 2023 15:58
parthea merged commit 7dfc3a7 into main Jun 12, 2023
parthea deleted the add-actionable-errors-lro branch June 12, 2023 16:00
This was referenced May 30, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL