Skip to content

Say why rdc open timed out, and don't retry a timeout - #277

Merged
BANANASJIM merged 1 commit into
BANANASJIM:masterfrom
VelocityRa:open-timeout-message
Oct 8, 2026
Merged

BANANASJIM merged 1 commit into
BANANASJIM:masterfrom
VelocityRa:open-timeout-message

Conversation

@VelocityRa

Copy link
Copy Markdown
Contributor

The open error now leads with the real reason (timeout) instead of the GPU fallback note. A timeout is no longer retried 3 times, which tripled the wait.

The error now leads with the real reason (timeout) instead of the GPU fallback note.

A timeout is no longer retried 3 times, which tripled the wait.
@VelocityRa
VelocityRa requested a review from BANANASJIM as a code owner October 8, 2026 00:06
@qodo-code-review

Copy link
Copy Markdown

ⓘ Qodo reviews are paused because the subscription is no longer active. Ask your workspace admin to reactivate the subscription to resume reviews. Manage billing

@coderabbitai

coderabbitai Bot commented Oct 8, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 23 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 2 included reviews currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: ab0ae2a4-2ef0-49e8-a973-6a2f3cd51e4e
📥 Commits

Reviewing files that changed from the base of the PR and between ae559ee and 00a7d75.

📒 Files selected for processing (2)
  • src/rdc/services/session_service.py
  • tests/unit/test_session_service.py
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@BANANASJIM
BANANASJIM merged commit dd94da9 into BANANASJIM:master Oct 8, 2026
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants