Skip to content

test(effect-runtime): measure the startup cost on this host (diagnostic) - #7715

Closed
xiaoan17 wants to merge 1 commit into
loopx-project:mainfrom
xiaoan17:codex/measure-windows-effect-runtime-startup
Closed

xiaoan17 wants to merge 1 commit into
loopx-project:mainfrom
xiaoan17:codex/measure-windows-effect-runtime-startup

Conversation

@xiaoan17

Copy link
Copy Markdown
Contributor

Temporary diagnostic. The windows-powershell lane reports runtime_startup_timeout where the typed mutation_lock_timeout diagnostic is expected. The Python ready budget is 15s and the TypeScript mutation-lock budget is 5s; the missing number is what the runtime server actually costs to start on the host that runs the lane.

This prints the measurements and never asserts, so it cannot fail a lane. It will be closed once the numbers are read.

@xiaoan17
xiaoan17 force-pushed the codex/measure-windows-effect-runtime-startup branch from f24c05e to ac5fa61 Compare October 30, 2026 03:24
The Windows lane reports runtime_startup_timeout where the typed
mutation_lock_timeout diagnostic is expected. The Python ready budget is
15s and the TypeScript mutation-lock budget is 5s; the missing number is
what the runtime server actually costs to start on the host that runs the
lane.

The measurement lives in test_effect_runtime_integration.py because the
Windows lane runs an explicit file list. CI runs pytest without -s, so
the measurement is raised as a deliberate failure to make it readable
from the job log. This file is a diagnostic, not a qualification.

Signed-off-by: xiaoan17 <anbc1224@gmail.com>
@xiaoan17
xiaoan17 force-pushed the codex/measure-windows-effect-runtime-startup branch from ac5fa61 to e756216 Compare October 30, 2026 04:09
@xiaoan17

Copy link
Copy Markdown
Contributor Author

诊断完成,测量代码已撤除。结论发在 #7695 的评论里。

@xiaoan17 xiaoan17 closed this Oct 30, 2026
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