Skip to content

Commit 11d75a9

Browse files
[CI] ensure pyyaml is installed into the correct Python environment (vllm-project#11328)
### What this PR does / why we need it? - The resolve_nightly_tests.py script requires pyyaml but it was not pre-installed in the runner container, causing ModuleNotFoundError. - Added explicit pip3 install step in the authorize job. - Removed --user flag from the script's fallback install to avoid PYTHONPATH issues in root containers. - vLLM version: v0.23.0 - vLLM main: vllm-project/vllm@a30addc Signed-off-by: hfadzxy <starmoon_zhang@163.com>
1 parent 924bff3 commit 11d75a9

2 files changed

Lines changed: 5 additions & 1 deletion

File tree

.github/workflows/pr_nightly_command.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -100,6 +100,10 @@ jobs:
100100
if: steps.auth.outputs.is_authorized == 'false'
101101
run: exit 1
102102

103+
- name: Install Python dependencies
104+
if: steps.auth.outputs.is_authorized == 'true'
105+
run: pip3 install pyyaml -q
106+
103107
- name: Resolve test cases and branch
104108
id: resolve
105109
if: steps.auth.outputs.is_authorized == 'true'

.github/workflows/scripts/resolve_nightly_tests.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
try:
88
import yaml
99
except ImportError:
10-
subprocess.check_call(["pip3", "install", "pyyaml", "-q", "--user"])
10+
subprocess.check_call(["pip3", "install", "pyyaml", "-q"])
1111
import yaml
1212

1313

0 commit comments

Comments
 (0)