Skip to content

fix: stop spurious 'drivemount failed' on OAuth waits over 10s#11

Merged
sethtroisi merged 1 commit into
googlecolab:mainfrom
teeler:drivemount-timeout
May 13, 2026
Merged

fix: stop spurious 'drivemount failed' on OAuth waits over 10s#11
sethtroisi merged 1 commit into
googlecolab:mainfrom
teeler:drivemount-timeout

Conversation

@teeler
Copy link
Copy Markdown
Contributor

@teeler teeler commented May 13, 2026

The drivefs handshake blocks the kernel on input_request while the user OAuths in their browser. jupyter_kernel_client's default 10s wall-clock timeout fires during this silent stretch, raising TimeoutError mid-flow even though the mount has actually succeeded server-side.

Add an optional timeout= parameter to ColabRuntime.execute_code that is forwarded to both execute() and execute_interactive(). Pass timeout=600 (10 min) from the human-in-the-loop subcommands -- colab drivemount and colab auth -- via a shared INTERACTIVE_AUTOMATION_TIMEOUT_SEC constant. Non-interactive paths keep the upstream default since their continuous iopub traffic never trips the practical inactivity ceiling.

The drivefs handshake blocks the kernel on input_request while the user
OAuths in their browser. jupyter_kernel_client's default 10s wall-clock
timeout fires during this silent stretch, raising TimeoutError mid-flow
even though the mount has actually succeeded server-side.

Add an optional timeout= parameter to ColabRuntime.execute_code that is
forwarded to both execute() and execute_interactive(). Pass timeout=600
(10 min) from the human-in-the-loop subcommands -- colab drivemount and
colab auth -- via a shared INTERACTIVE_AUTOMATION_TIMEOUT_SEC constant.
Non-interactive paths keep the upstream default since their continuous
iopub traffic never trips the practical inactivity ceiling.
@teeler teeler requested a review from sethtroisi May 13, 2026 00:35
@sethtroisi sethtroisi merged commit 510115b into googlecolab:main May 13, 2026
2 checks passed
@teeler teeler deleted the drivemount-timeout branch May 13, 2026 19:19
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