Commit a51fced
authored
Migrate H100/A100 runners to OSDC (ARC) (#179)
* Migrate linux.aws.h100/a100 runners to OSDC (ARC) runners
Switch all linux.aws.h100* and linux.aws.a100 runner labels to their
OSDC/ARC equivalents. Labels follow the mapping in pytorch/pytorch
.github/arc.yaml, with the mt- (Meta multi-tenant) prefix that OSDC
production runners use:
linux.aws.a100 -> mt-l-x86iavx512-11-125-a100 (1 GPU)
linux.aws.h100 -> mt-l-x86iamx-22-225-h100 (1 GPU)
linux.aws.h100.4 -> mt-l-x86iamx-88-900-h100-4 (4 GPU)
linux.aws.h100.8 -> mt-l-bx86iamx-176-1800-h100-8 (8 GPU)
Files:
- generate_vllm_benchmark_matrix.py: TP_TO_RUNNER_MAPPING and
RUNNER_TO_PLATFORM_MAPPING get the full label rename. In PLATFORM_SKIPS
the skip tokens become the bare GPU-type 'h100'/'a100' so they remain a
substring of the OSDC names, preserving the 'skip the whole family'
behavior the substring matcher relies on (matters for h100, which has
1/4/8-GPU variants).
- vllm-ci-test.yml, vllm-profiling.yml, pytorch-bisect.yaml: runs-on /
runner choice updated.
- test fixture: expected runner values updated to the OSDC names.
The matrix output is unchanged except for the runner label strings
(verified: every model<->runner pairing is identical after the rename).
* Containerize vLLM CI test + profiling workflows for OSDC runners
OSDC/ARC runners are ephemeral pods with no Docker daemon, so the old
'docker run --gpus all + docker exec' pattern cannot work on them. Run the
vLLM CI image via the job-level container: key with options '--gpus all'
instead (the GPU is injected by the runner pod), matching pytorch/pytorch
_linux-test.yml (test-osdc) and pytorch/helion.
- vllm-ci-test.yml / vllm-profiling.yml: add an ubuntu-latest 'resolve-image'
pre-job that runs 'docker manifest inspect' (needs a daemon the pod lacks)
to pick the latest available vLLM CI image and pass it down as the
container image. Drop the GPU_FLAG/docker run/docker exec wrapper and the
/tmp/workspace bind-mount; run the scripts directly in the container.
- vllm-profiling.yml: assume the upload IAM role via OIDC before the S3
upload (ephemeral pods have no host instance role); pass the resolved
vLLM commit through as S3_HEAD_SHA.
- run_vllm_profiling.sh: use $GITHUB_WORKSPACE instead of the hardcoded
/tmp/workspace bind-mount path.
* Containerize pytorch-bisect.yaml mt- (OSDC) path in a CUDA devel container
The mt- runner is an ephemeral OSDC pod with no host CUDA toolchain, so build
PyTorch inside pytorch/pytorch:2.12.0-cuda13.0-cudnn9-devel (--gpus all) when
the mt- runner is selected; linux.dgx.b200 keeps the existing bare-host path
(conditional container via fromJSON('null')). CUDA_HOME points at the image's
/usr/local/cuda on the container path (run.sh requires it non-empty). Add a
git safe.directory step for the root-owned in-container checkout.
* Containerize vllm-benchmark + sglang benchmarks for OSDC (all devices)
Run every matrix device inside a job-level container: instead of the old
'docker run + docker exec' pattern, since OSDC/ARC pods have no Docker daemon.
- generate_vllm_benchmark_matrix.py: emit a per-entry 'device-name' so the
workflow can resolve the container image up front (regenerated the test
fixture, which also clears pre-existing config drift).
- set-parameters: resolve the upstream image on ubuntu-latest (which has a
daemon) via 'docker manifest inspect', then enrich every matrix entry with
container-image + device-appropriate container-options. sglang resolves per
image suffix (cuda / -cu128-b200 / -rocm630-mi30x) and skips non-cuda/rocm
devices instead of failing the whole matrix.
- benchmarks job: add container: { image, options }, drop the device probe
(device-name comes from the matrix) while keeping the runtime DEVICE_TYPE
detection, run the benchmark script natively, and assume the upload IAM
role via OIDC for all devices (no host instance role inside a pod).
chown is made sudo-optional for the in-container root user.
Flagged for CI validation: the per-model S3 'already benchmarked' dedup is
dropped (needs the runtime device-type before the container exists); the
rocm/hpu container options are best-effort; and --shm-size may be capped on
ARC pods.
* Revert benchmark/sglang containerization; keep migration minimal
The H100/A100 benchmark jobs do not actually run in this repo - they run
in pytorch/pytorch's .github/workflows/vllm-benchmark.yml, which consumes
the runner labels emitted by generate_vllm_benchmark_matrix.py. So this
repo's vllm-benchmark.yml / sglang-benchmark.yml never execute on the
migrated OSDC runners and do not need the job-level container rewrite.
Revert f6bc62f's four-file change:
- vllm-benchmark.yml / sglang-benchmark.yml: back to the docker run/exec
pattern (they run on b200 / rocm / cpu / hpu, all of which keep a Docker
daemon - not OSDC).
- generate_vllm_benchmark_matrix.py: drop the per-entry device-name field
(it only existed to drive the dropped containerization). The H100/A100
-> OSDC label rename is kept, since pytorch/pytorch needs it.
- test fixture: regenerated from the current benchmark configs (clears
pre-existing config drift) with the mt- labels and no device-name.
What remains is the minimal migration: the runner label rename plus the
containerization of the three workflows that DO run on the migrated
runners here (vllm-ci-test, vllm-profiling, pytorch-bisect).1 parent dc4d250 commit a51fced
7 files changed
Lines changed: 214 additions & 435 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
14 | 14 | | |
15 | 15 | | |
16 | 16 | | |
17 | | - | |
| 17 | + | |
18 | 18 | | |
19 | 19 | | |
20 | 20 | | |
| |||
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
28 | | - | |
| 28 | + | |
29 | 29 | | |
30 | 30 | | |
31 | 31 | | |
32 | 32 | | |
33 | 33 | | |
34 | 34 | | |
35 | | - | |
| 35 | + | |
36 | 36 | | |
37 | 37 | | |
38 | 38 | | |
39 | 39 | | |
40 | 40 | | |
41 | | - | |
| 41 | + | |
42 | 42 | | |
43 | 43 | | |
44 | 44 | | |
| |||
47 | 47 | | |
48 | 48 | | |
49 | 49 | | |
50 | | - | |
51 | | - | |
52 | | - | |
53 | | - | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
54 | 54 | | |
55 | 55 | | |
56 | 56 | | |
| |||
102 | 102 | | |
103 | 103 | | |
104 | 104 | | |
105 | | - | |
106 | | - | |
| 105 | + | |
| 106 | + | |
107 | 107 | | |
108 | 108 | | |
109 | 109 | | |
110 | 110 | | |
111 | 111 | | |
112 | | - | |
113 | | - | |
| 112 | + | |
| 113 | + | |
114 | 114 | | |
115 | 115 | | |
116 | 116 | | |
117 | 117 | | |
118 | 118 | | |
119 | | - | |
120 | | - | |
| 119 | + | |
| 120 | + | |
121 | 121 | | |
122 | 122 | | |
123 | 123 | | |
124 | 124 | | |
125 | 125 | | |
126 | | - | |
127 | | - | |
| 126 | + | |
| 127 | + | |
128 | 128 | | |
129 | 129 | | |
130 | 130 | | |
131 | 131 | | |
132 | 132 | | |
133 | 133 | | |
134 | | - | |
| 134 | + | |
135 | 135 | | |
136 | 136 | | |
137 | 137 | | |
138 | 138 | | |
139 | | - | |
| 139 | + | |
140 | 140 | | |
141 | 141 | | |
142 | 142 | | |
143 | 143 | | |
144 | 144 | | |
145 | | - | |
146 | | - | |
| 145 | + | |
| 146 | + | |
147 | 147 | | |
148 | 148 | | |
149 | 149 | | |
150 | 150 | | |
151 | | - | |
152 | | - | |
| 151 | + | |
| 152 | + | |
153 | 153 | | |
154 | 154 | | |
155 | 155 | | |
156 | 156 | | |
157 | | - | |
| 157 | + | |
158 | 158 | | |
159 | | - | |
| 159 | + | |
160 | 160 | | |
161 | 161 | | |
162 | 162 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
15 | | - | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
16 | 18 | | |
17 | 19 | | |
18 | 20 | | |
| |||
0 commit comments