Unverified Commit c536bf76 authored by Tushar Sharma's avatar Tushar Sharma Committed by GitHub
Browse files

ci: remove core from deploy-operator & test trigger conditions (#6925)


Signed-off-by: default avatarTushar Sharma <tusharma@nvidia.com>
Co-authored-by: default avatarClaude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: default avatarAnant Sharma <anants@nvidia.com>
parent 97ef59dc
...@@ -10,7 +10,7 @@ When you open a PR, CI checks which files changed and runs only relevant jobs: ...@@ -10,7 +10,7 @@ When you open a PR, CI checks which files changed and runs only relevant jobs:
|--------|----------| |--------|----------|
| `core` | Main test suite (vLLM, SGLang, TRT-LLM containers) | | `core` | Main test suite (vLLM, SGLang, TRT-LLM containers) |
| `operator` | Kubernetes operator tests | | `operator` | Kubernetes operator tests |
| `deploy` | Helm chart validation | | `deploy` | Deploy-specific tests |
| `vllm` / `sglang` / `trtllm` | Backend-specific tests | | `vllm` / `sglang` / `trtllm` | Backend-specific tests |
| `docs` | Nothing (classification only) | | `docs` | Nothing (classification only) |
| `examples` | Nothing (classification only) | | `examples` | Nothing (classification only) |
......
# CI Filter Patterns - See FILTERS.md for documentation # CI Filter Patterns - See FILTERS.md for documentation
# #
# Filters that trigger CI jobs: # Filters that trigger CI jobs:
# core -> dynamo build-test, all backend builds (vllm/sglang/trtllm), deploy tests # core -> dynamo build-test, all backend builds (vllm/sglang/trtllm)
# operator -> operator build and test # operator -> operator build and test
# deploy -> deploy-test-vllm-disagg-router # deploy -> deploy tests
# planner -> dynamo build-test # planner -> dynamo build-test
# vllm -> vllm build and test # vllm -> vllm build and test
# sglang -> sglang build and test # sglang -> sglang build and test
...@@ -105,9 +105,11 @@ operator: ...@@ -105,9 +105,11 @@ operator:
deploy: deploy:
- '!**/*.md' - '!**/*.md'
- '!**/*.rst' - '!**/*.rst'
- *ci
- 'deploy/helm/**' - 'deploy/helm/**'
- 'deploy/utils/**' - 'deploy/utils/**'
- 'deploy/chrek/**' - 'deploy/chrek/**'
- 'tests/deploy/**'
planner: planner:
- 'components/src/dynamo/planner/**' - 'components/src/dynamo/planner/**'
......
...@@ -181,7 +181,7 @@ jobs: ...@@ -181,7 +181,7 @@ jobs:
# ============================================================================ # ============================================================================
vllm-pipeline: vllm-pipeline:
needs: [changed-files] needs: [changed-files]
if: needs.changed-files.outputs.core == 'true' || needs.changed-files.outputs.vllm == 'true' if: needs.changed-files.outputs.core == 'true' || needs.changed-files.outputs.vllm == 'true' || needs.changed-files.outputs.deploy == 'true'
uses: ./.github/workflows/build-test-distribute-flavor-matrix.yml uses: ./.github/workflows/build-test-distribute-flavor-matrix.yml
with: with:
framework: vllm framework: vllm
...@@ -191,7 +191,9 @@ jobs: ...@@ -191,7 +191,9 @@ jobs:
builder_name: ${{ needs.changed-files.outputs.builder_name }} builder_name: ${{ needs.changed-files.outputs.builder_name }}
build_timeout_minutes: 60 build_timeout_minutes: 60
copy_timeout_minutes: 10 copy_timeout_minutes: 10
run_cpu_only_tests: ${{ needs.changed-files.outputs.core == 'true' || needs.changed-files.outputs.vllm == 'true' }}
cpu_only_test_markers: 'pre_merge and vllm and gpu_0' cpu_only_test_markers: 'pre_merge and vllm and gpu_0'
run_single_gpu_tests: ${{ needs.changed-files.outputs.core == 'true' || needs.changed-files.outputs.vllm == 'true' }}
single_gpu_test_markers: 'pre_merge and vllm and gpu_1' single_gpu_test_markers: 'pre_merge and vllm and gpu_1'
single_gpu_test_timeout_minutes: 35 single_gpu_test_timeout_minutes: 35
run_multi_gpu_tests: false # TODO: Dmitry is working on fixing markers for multi-GPU tests, can enable after that is resolved run_multi_gpu_tests: false # TODO: Dmitry is working on fixing markers for multi-GPU tests, can enable after that is resolved
...@@ -202,7 +204,7 @@ jobs: ...@@ -202,7 +204,7 @@ jobs:
# ============================================================================ # ============================================================================
sglang-pipeline: sglang-pipeline:
needs: [changed-files] needs: [changed-files]
if: needs.changed-files.outputs.core == 'true' || needs.changed-files.outputs.sglang == 'true' if: needs.changed-files.outputs.core == 'true' || needs.changed-files.outputs.sglang == 'true' || needs.changed-files.outputs.deploy == 'true'
uses: ./.github/workflows/build-test-distribute-flavor-matrix.yml uses: ./.github/workflows/build-test-distribute-flavor-matrix.yml
with: with:
framework: sglang framework: sglang
...@@ -212,7 +214,9 @@ jobs: ...@@ -212,7 +214,9 @@ jobs:
builder_name: ${{ needs.changed-files.outputs.builder_name }} builder_name: ${{ needs.changed-files.outputs.builder_name }}
build_timeout_minutes: 60 build_timeout_minutes: 60
copy_timeout_minutes: 10 copy_timeout_minutes: 10
run_cpu_only_tests: ${{ needs.changed-files.outputs.core == 'true' || needs.changed-files.outputs.sglang == 'true' }}
cpu_only_test_markers: 'pre_merge and sglang and gpu_0' cpu_only_test_markers: 'pre_merge and sglang and gpu_0'
run_single_gpu_tests: ${{ needs.changed-files.outputs.core == 'true' || needs.changed-files.outputs.sglang == 'true' }}
single_gpu_test_markers: 'pre_merge and sglang and gpu_1' single_gpu_test_markers: 'pre_merge and sglang and gpu_1'
run_multi_gpu_tests: false # TODO: Dmitry is working on fixing markers for multi-GPU tests, can enable after that is resolved run_multi_gpu_tests: false # TODO: Dmitry is working on fixing markers for multi-GPU tests, can enable after that is resolved
secrets: inherit secrets: inherit
...@@ -222,7 +226,7 @@ jobs: ...@@ -222,7 +226,7 @@ jobs:
# ============================================================================ # ============================================================================
trtllm-pipeline: trtllm-pipeline:
needs: [changed-files] needs: [changed-files]
if: needs.changed-files.outputs.core == 'true' || needs.changed-files.outputs.trtllm == 'true' if: needs.changed-files.outputs.core == 'true' || needs.changed-files.outputs.trtllm == 'true' || needs.changed-files.outputs.deploy == 'true'
uses: ./.github/workflows/build-test-distribute-flavor-matrix.yml uses: ./.github/workflows/build-test-distribute-flavor-matrix.yml
with: with:
framework: trtllm framework: trtllm
...@@ -232,7 +236,9 @@ jobs: ...@@ -232,7 +236,9 @@ jobs:
builder_name: ${{ needs.changed-files.outputs.builder_name }} builder_name: ${{ needs.changed-files.outputs.builder_name }}
build_timeout_minutes: 60 build_timeout_minutes: 60
copy_timeout_minutes: 10 copy_timeout_minutes: 10
run_cpu_only_tests: ${{ needs.changed-files.outputs.core == 'true' || needs.changed-files.outputs.trtllm == 'true' }}
cpu_only_test_markers: 'pre_merge and trtllm and gpu_0' cpu_only_test_markers: 'pre_merge and trtllm and gpu_0'
run_single_gpu_tests: ${{ needs.changed-files.outputs.core == 'true' || needs.changed-files.outputs.trtllm == 'true' }}
single_gpu_test_markers: 'pre_merge and trtllm and gpu_1' single_gpu_test_markers: 'pre_merge and trtllm and gpu_1'
run_multi_gpu_tests: false # TODO: Dmitry is working on fixing markers for multi-GPU tests, can enable after that is resolved run_multi_gpu_tests: false # TODO: Dmitry is working on fixing markers for multi-GPU tests, can enable after that is resolved
secrets: inherit secrets: inherit
...@@ -245,11 +251,10 @@ jobs: ...@@ -245,11 +251,10 @@ jobs:
deploy-operator: deploy-operator:
runs-on: prod-default-small-v2 runs-on: prod-default-small-v2
# Run when any deploy test will run: if core, any framework, or deploy files changed # Run when any deploy test will run: if any framework or deploy files changed
if: | if: |
always() && always() &&
(needs.changed-files.outputs.core == 'true' || (needs.changed-files.outputs.vllm == 'true' ||
needs.changed-files.outputs.vllm == 'true' ||
needs.changed-files.outputs.sglang == 'true' || needs.changed-files.outputs.sglang == 'true' ||
needs.changed-files.outputs.trtllm == 'true' || needs.changed-files.outputs.trtllm == 'true' ||
needs.changed-files.outputs.deploy == 'true') && needs.changed-files.outputs.deploy == 'true') &&
...@@ -302,7 +307,7 @@ jobs: ...@@ -302,7 +307,7 @@ jobs:
# jobs unless always() is present. !failure() ensures we still skip on real failures. # jobs unless always() is present. !failure() ensures we still skip on real failures.
if: | if: |
always() && !failure() && always() && !failure() &&
(needs.changed-files.outputs.core == 'true' || needs.changed-files.outputs.vllm == 'true' || needs.changed-files.outputs.deploy == 'true') (needs.changed-files.outputs.vllm == 'true' || needs.changed-files.outputs.deploy == 'true')
runs-on: prod-default-small-v2 runs-on: prod-default-small-v2
needs: [changed-files, deploy-operator, vllm-pipeline] needs: [changed-files, deploy-operator, vllm-pipeline]
timeout-minutes: 25 timeout-minutes: 25
...@@ -342,7 +347,7 @@ jobs: ...@@ -342,7 +347,7 @@ jobs:
# Run if core, sglang, or deploy is changed (see deploy-test-vllm for always() rationale) # Run if core, sglang, or deploy is changed (see deploy-test-vllm for always() rationale)
if: | if: |
always() && !failure() && always() && !failure() &&
(needs.changed-files.outputs.core == 'true' || needs.changed-files.outputs.sglang == 'true' || needs.changed-files.outputs.deploy == 'true') (needs.changed-files.outputs.sglang == 'true' || needs.changed-files.outputs.deploy == 'true')
needs: [changed-files, deploy-operator, sglang-pipeline] needs: [changed-files, deploy-operator, sglang-pipeline]
timeout-minutes: 25 timeout-minutes: 25
permissions: permissions:
...@@ -379,7 +384,7 @@ jobs: ...@@ -379,7 +384,7 @@ jobs:
# Run if core, trtllm, or deploy is changed (see deploy-test-vllm for always() rationale) # Run if core, trtllm, or deploy is changed (see deploy-test-vllm for always() rationale)
if: | if: |
always() && !failure() && always() && !failure() &&
(needs.changed-files.outputs.core == 'true' || needs.changed-files.outputs.trtllm == 'true' || needs.changed-files.outputs.deploy == 'true') (needs.changed-files.outputs.trtllm == 'true' || needs.changed-files.outputs.deploy == 'true')
needs: [changed-files, deploy-operator, trtllm-pipeline] needs: [changed-files, deploy-operator, trtllm-pipeline]
timeout-minutes: 25 timeout-minutes: 25
permissions: permissions:
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment