Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
parthea merged 8 commits intomainfromadd-actionable-errors-lro
Jun 12, 2023

Conversation

@parthea
Copy link
Collaborator

@partheaparthea commentedMay 19, 2023•
edited
Loading

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 REQUESTINCOMPATIBLE_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: sPull request size is small. labelMay 19, 2023
@partheapartheaforce-pushed theadd-actionable-errors-lro branch 2 times, most recently from50b5a79 to254e151CompareMay 19, 2023 17:04
@partheapartheaforce-pushed theadd-actionable-errors-lro branch from254e151 to0e3f9bdCompareMay 19, 2023 18:44
Copy link
CollaboratorAuthor

@partheaparthea left a comment

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: mPull request size is medium. and removed size: sPull request size is small. labelsJun 1, 2023
@partheaparthea marked this pull request as ready for reviewJune 1, 2023 14:34
@partheaparthea requested review froma team ascode ownersJune 1, 2023 14:34
@partheapartheaenabled auto-merge (squash)June 12, 2023 15:58
@partheaparthea merged commit7dfc3a7 intomainJun 12, 2023
@partheaparthea deleted the add-actionable-errors-lro branchJune 12, 2023 16:00
This was referencedMay 30, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@vchudnov-gvchudnov-gvchudnov-g approved these changes

@dizcologydizcologyAwaiting requested review from dizcology

Labels

size: mPull request size is medium.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@parthea@vchudnov-g@dizcology
[8]ページ先頭
©2009-2025 Movatter.jp