fix(e2e): use --active to skip per-project venv sync in provisioner#329
Merged
runpod-Henrik merged 5 commits intomainfrom Apr 28, 2026
Merged
fix(e2e): use --active to skip per-project venv sync in provisioner#329runpod-Henrik merged 5 commits intomainfrom
runpod-Henrik merged 5 commits intomainfrom
Conversation
uv run without --active creates a project-specific venv from the temp dir's pyproject.toml, which contains runpod-flash @ git+https://... The private repo fetch fails on CI runners that have no git credentials. The outer CI venv (uv sync --all-groups) already has the correct flash version installed, so --active uses it directly. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
deanq
approved these changes
Apr 28, 2026
--active was not sufficient: uv still reads the pyproject.toml and tries to sync the git dep into the active env, hitting the same auth wall. --no-project skips project discovery entirely so uv uses the outer CI venv (which already has the correct flash installed) without touching the private git dep. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Same fix as provisioner.py: all deploy/undeploy calls in test files run in temp dirs with a pyproject.toml that references the private git dep, causing uv to attempt an authenticated fetch before executing the command. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…release provisioner.py: --no-project skips the temp project venv, so extra_deps (numpy, pandas) aren't installed when flash validates the worker import. Pass each extra dep as --with <dep> so uv injects them into the run env. test_rolling_release.py: the "updating endpoint" check reads from log.debug() in serverless.py. LOG_LEVEL=INFO silences it; switch to DEBUG. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
provisioner.pyranuv run flash deployin a temp project dir whosepyproject.tomldeclaredrunpod-flash @ git+https://github.com/runpod/runpod-flash@<sha>uvtried to sync that dep before running the command, hitting the private repo with no git credentials →fatal: could not read Usernameuv sync --all-groups) already has the correct flash version installedFix
Use
--no-projectso uv skips project discovery entirely and uses the outer CI venv without touching the private git dep. (--activealone is not sufficient — uv still reads the pyproject.toml and tries to sync into the active env.)Test plan
Re-run the E2E workflow — the provisioner deploy step should no longer fail with a git auth error.
🤖 Generated with Claude Code