Skip to content
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.

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Add UNKNOWN status code to retryable Quantum Engine errors #6565

Merged
merged 5 commits into from
Apr 18, 2024
Merged

Add UNKNOWN status code to retryable Quantum Engine errors #6565

merged 5 commits into from
Apr 18, 2024

Conversation

wcourtney
Copy link
Collaborator

Users have reported intermittent errors where requests are halted due to a timeout. Within a provided stacktrace, I found:

<AioRpcError of RPC that terminated with:
	status = StatusCode.UNKNOWN
	details = "408:Request Timeout"
	debug_error_string = "<...>"
>

The stack trace also indicated that the _is_retryable_error check failed and so raised the exception.

This change adds the UNKNOWN error code to retry in these cases.

@wcourtney wcourtney requested review from verult and senecameeks April 18, 2024 01:46
@wcourtney wcourtney marked this pull request as ready for review April 18, 2024 01:49
@wcourtney wcourtney requested review from vtomole, cduck and a team as code owners April 18, 2024 01:49
@wcourtney wcourtney merged commit 21986dd into main Apr 18, 2024
32 checks passed
@wcourtney wcourtney deleted the u/wcourtney/nocancel branch April 18, 2024 17:11
jselig-rigetti pushed a commit to jselig-rigetti/Cirq that referenced this pull request May 28, 2024
@wcourtney @jselig-rigetti
Add UNKNOWN status code to retryable Quantum Engine errors (quantumli…
b4bbb02 
…b#6565)

* Add UNKNOWN status code to retryable errors.

* Update test

* Revert unintended changes.

* More removals.

* Lint formatting
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pavoljuhas pavoljuhas pavoljuhas approved these changes

@verult verult Awaiting requested review from verult verult is a code owner

@senecameeks senecameeks Awaiting requested review from senecameeks

@vtomole vtomole Awaiting requested review from vtomole vtomole is a code owner

@cduck cduck Awaiting requested review from cduck

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@wcourtney @pavoljuhas