Skip to content

test: stabilize notebook queue concurrency gate - #285

Merged
Ishaan Gangwani (ishaan1124) merged 1 commit into
mainfrom
codex/release-notebook-queue-gate
Aug 14, 2026
Merged

test: stabilize notebook queue concurrency gate#285
Ishaan Gangwani (ishaan1124) merged 1 commit into
mainfrom
codex/release-notebook-queue-gate

Conversation

@ishaan1124

Copy link
Copy Markdown
Member

Summary

  • replace fixed 20 ms sleeps with session-workspace release sentinels
  • observe first-running, second-queued, then second-running through the public status route
  • bound both Python waits and the scenario deadline, with guaranteed release and kernel cleanup

Root cause

Kernel execution durably records the submitted cell before joining KernelQueue. Under full-suite load, the second HTTP request had not completed that durable admission within 20 ms, so queue_depth 0 was valid even though serialization remained correct.

Validation

  • pre-fix load reproduced 3 failures in 12 fresh processes
  • final Linux Bun 1.3.14 stress: 12 of 12 pass
  • post-fix load: 24 of 24 pass
  • full notebook route file: 31 pass, 0 fail
  • backend and root typechecks, Prettier, diff check, and staged secret scan pass
  • independent review: no P0 or P1 blocker

Fixes the sole failure in exact-main CI run 31780153018.

@vercel

vercel Bot commented Aug 14, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openscience Ready Ready Preview Aug 14, 2026 8:06am

Request Review

@ishaan1124
Ishaan Gangwani (ishaan1124) merged commit 2da0409 into main Aug 14, 2026
17 checks passed
@ishaan1124
Ishaan Gangwani (ishaan1124) deleted the codex/release-notebook-queue-gate branch August 14, 2026 08:17
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.

1 participant