Skip to content

Fix UV benchmark pipeline: Linux PATH, venv dot-source params, and install pattern#1148

Draft
Copilot wants to merge 2 commits intousers/pahallis/run-benchfrom
copilot/sub-pr-1112
Draft

Fix UV benchmark pipeline: Linux PATH, venv dot-source params, and install pattern#1148
Copilot wants to merge 2 commits intousers/pahallis/run-benchfrom
copilot/sub-pr-1112

Conversation

Copy link
Contributor

Copilot AI commented Mar 5, 2026

Three bugs in the UV-based benchmark pipeline prevented it from running successfully on Linux CI agents.

Bugs fixed

  • install-uv.yml — Linux PATH never updated: $HOME/.local/bin prepend step was conditioned on Darwin only; the CI pool is Linux, so uv was installed but not on PATH. Condition expanded to or(Linux, Darwin).

  • Invoke-CopilotBenchmarks.ps1 — broken venv dot-source: Script dot-sourced Create-Venv.ps1 / Activate-Venv.ps1 passing -RepoRoot, which neither script's param() block declares → immediate runtime error. Removed the dead dot-source calls and unused $repoRoot variable.

  • Invoke-CopilotBenchmarks.ps1 — wrong UV install/run pattern: uv pip install requires an active venv or --system; uv run 'msbench-cli' doesn't invoke an installed CLI tool. Replaced with the correct UV-native pattern:

# Before (broken)
uv pip install msbench-cli
uv run 'msbench-cli' version
uv run 'msbench-cli' @runArgs

# After
uv tool install msbench-cli   # isolated env, binary landed in ~/.local/bin (now on PATH)
msbench-cli version
msbench-cli @runArgs

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…e, use uv tool install

Co-authored-by: hallipr <1291634+hallipr@users.noreply.github.com>
Copilot AI changed the title [WIP] Use UV for faster builds in Azure benchmark pipeline Fix UV benchmark pipeline: Linux PATH, venv dot-source params, and install pattern Mar 5, 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.

2 participants