Skip to content

Add UNKNOWN status code to retryable Quantum Engine errors #6565

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
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 senecameeks and verult 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 a team, cduck and vtomole as code owners April 18, 2024 01:49
@wcourtney wcourtney merged commit 21986dd into main Apr 18, 2024
@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
BichengYing pushed a commit to BichengYing/Cirq that referenced this pull request Jun 20, 2025
@wcourtney @BichengYing
Add UNKNOWN status code to retryable Quantum Engine errors (quantumli…
3c21840 
…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.

3 participants

@wcourtney @pavoljuhas