Skip to content

Commit 5da0bb0

Browse files
authored
Merge branch 'main' into pvijayakrish/nightly-ci-workflow
2 parents 52fb615 + db5687f commit 5da0bb0

File tree

147 files changed

+5557
-1290
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

147 files changed

+5557
-1290
lines changed

.github/workflows/container-validation-backends.yml

Lines changed: 9 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -178,23 +178,14 @@ jobs:
178178
azure_acr_user: ${{ secrets.AZURE_ACR_USER }}
179179
azure_acr_password: ${{ secrets.AZURE_ACR_PASSWORD }}
180180

181-
- name: Run unit tests
181+
- name: Run tests
182182
if: ${{ matrix.platform.arch != 'arm64' }}
183183
uses: ./.github/actions/pytest
184184
with:
185185
image_tag: ${{ steps.build-image.outputs.image_tag }}
186-
pytest_marks: "unit and vllm and gpu_1"
186+
pytest_marks: "pre_merge and vllm"
187187
framework: "vllm"
188-
test_type: "unit"
189-
platform_arch: ${{ matrix.platform.arch }}
190-
- name: Run e2e tests
191-
if: ${{ matrix.platform.arch != 'arm64' }}
192-
uses: ./.github/actions/pytest
193-
with:
194-
image_tag: ${{ steps.build-image.outputs.image_tag }}
195-
pytest_marks: "e2e and vllm and gpu_1 and not slow"
196-
framework: "vllm"
197-
test_type: "e2e, gpu_1"
188+
test_type: "pre_merge"
198189
platform_arch: ${{ matrix.platform.arch }}
199190

200191
sglang:
@@ -245,23 +236,14 @@ jobs:
245236
azure_acr_user: ${{ secrets.AZURE_ACR_USER }}
246237
azure_acr_password: ${{ secrets.AZURE_ACR_PASSWORD }}
247238

248-
- name: Run unit tests
249-
if: ${{ matrix.platform.arch != 'arm64' }}
250-
uses: ./.github/actions/pytest
251-
with:
252-
image_tag: ${{ steps.build-image.outputs.image_tag }}
253-
pytest_marks: "unit and sglang and gpu_1"
254-
framework: "sglang"
255-
test_type: "unit"
256-
platform_arch: ${{ matrix.platform.arch }}
257-
- name: Run e2e tests
239+
- name: Run tests
258240
if: ${{ matrix.platform.arch != 'arm64' }}
259241
uses: ./.github/actions/pytest
260242
with:
261243
image_tag: ${{ steps.build-image.outputs.image_tag }}
262-
pytest_marks: "e2e and sglang and gpu_1"
244+
pytest_marks: "pre_merge and sglang"
263245
framework: "sglang"
264-
test_type: "e2e, gpu_1"
246+
test_type: "pre_merge"
265247
platform_arch: ${{ matrix.platform.arch }}
266248

267249
trtllm:
@@ -312,23 +294,14 @@ jobs:
312294
azure_acr_user: ${{ secrets.AZURE_ACR_USER }}
313295
azure_acr_password: ${{ secrets.AZURE_ACR_PASSWORD }}
314296

315-
- name: Run unit tests
316-
if: ${{ matrix.platform.arch != 'arm64' }}
317-
uses: ./.github/actions/pytest
318-
with:
319-
image_tag: ${{ steps.build-image.outputs.image_tag }}
320-
pytest_marks: "unit and trtllm and gpu_1"
321-
framework: "trtllm"
322-
test_type: "unit"
323-
platform_arch: ${{ matrix.platform.arch }}
324-
- name: Run e2e tests
297+
- name: Run tests
325298
if: ${{ matrix.platform.arch != 'arm64' }}
326299
uses: ./.github/actions/pytest
327300
with:
328301
image_tag: ${{ steps.build-image.outputs.image_tag }}
329-
pytest_marks: "e2e and trtllm and gpu_1 and not slow"
302+
pytest_marks: "pre_merge and trtllm"
330303
framework: "trtllm"
331-
test_type: "e2e, gpu_1"
304+
test_type: "pre_merge"
332305
platform_arch: ${{ matrix.platform.arch }}
333306

334307
deploy-test-fault-tolerance:

.github/workflows/container-validation-dynamo.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ jobs:
6666
docker compose down
6767
- name: Run pytest (parallel tests with xdist)
6868
env:
69-
PYTEST_MARKS: "pre_merge and parallel"
69+
PYTEST_MARKS: "pre_merge and parallel and not (vllm or sglang or trtllm)"
7070
run: |
7171
docker run -w /workspace \
7272
--name ${{ env.CONTAINER_ID }}_pytest_parallel \
@@ -78,7 +78,7 @@ jobs:
7878
docker cp ${{ env.CONTAINER_ID }}_pytest_parallel:/workspace/${{ env.PYTEST_PARALLEL_XML_FILE }} . || echo "No parallel test report found"
7979
- name: Run pytest (sequential tests)
8080
env:
81-
PYTEST_MARKS: "(pre_merge and not parallel) or mypy"
81+
PYTEST_MARKS: "((pre_merge and not parallel) or mypy) and not (vllm or sglang or trtllm)"
8282
run: |
8383
docker run -w /workspace \
8484
--name ${{ env.CONTAINER_ID }}_pytest \

.gitignore

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -77,6 +77,9 @@ llm_engine.h
7777
### Ruff ###
7878
.ruff_cache/
7979

80+
### MyPy ###
81+
.mypy_cache/
82+
8083
### Python ###
8184
__pycache__/
8285
*.py[cod]

0 commit comments

Comments
 (0)