Skip to content

chore: bump verifiers (race-free uv script cache) - #3897

Closed
faresobeid wants to merge 1 commit into
mainfrom
chore/bump-verifiers-uv-script-fix
Closed

faresobeid wants to merge 1 commit into
mainfrom
chore/bump-verifiers-uv-script-fix

Conversation

@faresobeid

@faresobeid faresobeid commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Moves the deps/verifiers submodule from 4fd5921 to 1dbd62d. That commit is the current pin 4fd5921 plus only the fix from PrimeIntellect-ai/verifiers#2777, cherry-picked onto verifiers branch fix/uv-script-race-on-4fd5921. No other verifiers changes come along.

Re-pin to the merge commit once verifiers#2777 lands.

Why

prepare_uv_script published scripts with mv -f <tmp> <path> into the shared ~/.cache/verifiers/runtimes/scripts cache. That replaced the file while workers on other nodes were running it, which gave ENOENT on the shared FS. The fix publishes with ln (link-if-absent), so an existing script is never replaced.

Checks

pytest tests/unit/orchestrator tests/unit/test_configs.py -m "not gpu" with verifiers at 1dbd62d: 287 passed. uv.lock does not change.

@faresobeid
faresobeid force-pushed the chore/bump-verifiers-uv-script-fix branch from 505a467 to 0c037f9 Compare October 6, 2026 11:44
Base automatically changed from chore/bump-vllm-0.31 to main October 6, 2026 14:46
@faresobeid
faresobeid force-pushed the chore/bump-verifiers-uv-script-fix branch from 0c037f9 to 99a3672 Compare October 6, 2026 14:53
@faresobeid

Copy link
Copy Markdown
Contributor Author

Closing: the verifiers pin will be moved to verifiers main in #3900 once verifiers#2777 (race-free uv script cache) and #2778 merge, so a separate side-branch pin isn't needed.

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