Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
47 changes: 0 additions & 47 deletions .github/actions/check-ci/action.yml

This file was deleted.

16 changes: 1 addition & 15 deletions .github/workflows/benchmark-manual-policy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,23 +33,9 @@ permissions:
contents: read

jobs:
check-ci:
runs-on: ubuntu-latest
outputs:
should_run: ${{ steps.check.outputs.should_run }}
steps:
- uses: actions/checkout@v6
with:
sparse-checkout: .github/actions/check-ci
- id: check
uses: ./.github/actions/check-ci

benchmark:
name: Manual Policy (Sticky Sessions)
needs: check-ci
if: |
github.repository == 'lightseekorg/smg' &&
needs.check-ci.outputs.should_run == 'true'
if: github.repository == 'lightseekorg/smg'
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v6
Expand Down
16 changes: 1 addition & 15 deletions .github/workflows/benchmark-radix-tree.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,23 +33,9 @@ permissions:
contents: read

jobs:
check-ci:
runs-on: ubuntu-latest
outputs:
should_run: ${{ steps.check.outputs.should_run }}
steps:
- uses: actions/checkout@v6
with:
sparse-checkout: .github/actions/check-ci
- id: check
uses: ./.github/actions/check-ci

benchmark:
name: Radix Tree (Cache-Aware Routing)
needs: check-ci
if: |
github.repository == 'lightseekorg/smg' &&
needs.check-ci.outputs.should_run == 'true'
if: github.repository == 'lightseekorg/smg'
runs-on: k8s-runner-gpu
steps:
- uses: actions/checkout@v6
Expand Down
16 changes: 1 addition & 15 deletions .github/workflows/benchmark-request-processing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,23 +37,9 @@ permissions:
contents: read

jobs:
check-ci:
runs-on: ubuntu-latest
outputs:
should_run: ${{ steps.check.outputs.should_run }}
steps:
- uses: actions/checkout@v6
with:
sparse-checkout: .github/actions/check-ci
- id: check
uses: ./.github/actions/check-ci

benchmark:
name: Request Processing
needs: check-ci
if: |
github.repository == 'lightseekorg/smg' &&
needs.check-ci.outputs.should_run == 'true'
if: github.repository == 'lightseekorg/smg'
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v6
Expand Down
16 changes: 1 addition & 15 deletions .github/workflows/benchmark-tokenizer.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,23 +33,9 @@ permissions:
contents: read

jobs:
check-ci:
runs-on: ubuntu-latest
outputs:
should_run: ${{ steps.check.outputs.should_run }}
steps:
- uses: actions/checkout@v6
with:
sparse-checkout: .github/actions/check-ci
- id: check
uses: ./.github/actions/check-ci

benchmark:
name: Tokenizer
needs: check-ci
if: |
github.repository == 'lightseekorg/smg' &&
needs.check-ci.outputs.should_run == 'true'
if: github.repository == 'lightseekorg/smg'
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v6
Expand Down
16 changes: 1 addition & 15 deletions .github/workflows/benchmark-tool-parser.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,23 +33,9 @@ permissions:
contents: read

jobs:
check-ci:
runs-on: ubuntu-latest
outputs:
should_run: ${{ steps.check.outputs.should_run }}
steps:
- uses: actions/checkout@v6
with:
sparse-checkout: .github/actions/check-ci
- id: check
uses: ./.github/actions/check-ci

benchmark:
name: Tool Parser
needs: check-ci
if: |
github.repository == 'lightseekorg/smg' &&
needs.check-ci.outputs.should_run == 'true'
if: github.repository == 'lightseekorg/smg'
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v6
Expand Down
50 changes: 16 additions & 34 deletions .github/workflows/pr-test-rust.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ on:
- "*.md"
pull_request:
branches: [ main ]
types: [opened, synchronize, reopened, labeled]
types: [opened, synchronize, reopened]
paths-ignore:
- "docs/**"
- "mkdocs.yml"
Expand All @@ -29,22 +29,7 @@ env:
GENAI_BENCH_IMAGE: ghcr.io/moirai-internal/genai-bench:0.0.3

jobs:
check-ci:
runs-on: k8s-runner-cpu
permissions:
contents: read
outputs:
should_run: ${{ steps.check.outputs.should_run }}
steps:
- uses: actions/checkout@v6
with:
sparse-checkout: .github/actions/check-ci
- id: check
uses: ./.github/actions/check-ci

python-lint:
needs: check-ci
if: needs.check-ci.outputs.should_run == 'true'
runs-on: k8s-runner-cpu
permissions:
contents: read
Expand Down Expand Up @@ -72,8 +57,6 @@ jobs:
run: mypy bindings/python/ --config-file mypy.ini

grpc-proto-build-check:
needs: check-ci
if: needs.check-ci.outputs.should_run == 'true'
runs-on: k8s-runner-cpu
permissions:
contents: read
Expand All @@ -100,8 +83,6 @@ jobs:
python -c "from smg_grpc_proto import sglang_scheduler_pb2; print('OK')"

build-wheel:
needs: check-ci
if: needs.check-ci.outputs.should_run == 'true'
runs-on: k8s-runner-gpu
permissions:
contents: read
Expand Down Expand Up @@ -204,8 +185,6 @@ jobs:
pytest -q tests --cov=smg --cov-config=.coveragerc --cov-report=term-missing --cov-fail-under=80

unit-tests:
needs: check-ci
if: needs.check-ci.outputs.should_run == 'true'
runs-on: k8s-runner-cpu
permissions:
contents: read
Expand Down Expand Up @@ -257,8 +236,8 @@ jobs:

gateway-e2e:
name: ${{ matrix.name }}
needs: [check-ci, build-wheel]
if: needs.check-ci.outputs.should_run == 'true'
needs: build-wheel
if: github.actor != 'dependabot[bot]'
Comment thread
CatherineSue marked this conversation as resolved.
permissions:
contents: read
strategy:
Expand Down Expand Up @@ -411,8 +390,7 @@ jobs:

go-unit-tests:
name: go-unit-tests
needs: [check-ci, build-wheel]
if: needs.check-ci.outputs.should_run == 'true'
needs: build-wheel
runs-on: k8s-runner-cpu
timeout-minutes: 15
permissions:
Expand Down Expand Up @@ -452,8 +430,8 @@ jobs:

go-bindings-e2e:
name: go-bindings-e2e
needs: [check-ci, build-wheel]
if: needs.check-ci.outputs.should_run == 'true'
needs: build-wheel
if: github.actor != 'dependabot[bot]'
runs-on: k8s-runner-gpu
timeout-minutes: 20
permissions:
Expand Down Expand Up @@ -505,7 +483,7 @@ jobs:

go-bindings-benchmark:
name: go-bindings-benchmark
needs: [check-ci, build-wheel]
needs: build-wheel
if: false # Disabled
runs-on: k8s-runner-gpu
timeout-minutes: 32
Expand Down Expand Up @@ -568,17 +546,21 @@ jobs:
path: benchmark_go_bindings/

finish:
needs: [check-ci, python-lint, grpc-proto-build-check, build-wheel, python-unit-tests, unit-tests, gateway-e2e, go-unit-tests, go-bindings-e2e]
needs: [python-lint, grpc-proto-build-check, build-wheel, python-unit-tests, unit-tests, gateway-e2e, go-unit-tests, go-bindings-e2e]
if: always()
runs-on: k8s-runner-cpu
permissions: {}
steps:
- name: Check CI result
run: |
if [[ "${{ needs.check-ci.outputs.should_run }}" != "true" ]]; then
echo "CI was skipped (external contributor without run-ci label)"
exit 0
elif [[ "${{ needs.python-lint.result }}" == "failure" || "${{ needs.grpc-proto-build-check.result }}" == "failure" || "${{ needs.build-wheel.result }}" == "failure" || "${{ needs.python-unit-tests.result }}" == "failure" || "${{ needs.unit-tests.result }}" == "failure" || "${{ needs.gateway-e2e.result }}" == "failure" || "${{ needs.go-unit-tests.result }}" == "failure" || "${{ needs.go-bindings-e2e.result }}" == "failure" ]]; then
if [[ "${{ needs.python-lint.result }}" == "failure" || \
"${{ needs.grpc-proto-build-check.result }}" == "failure" || \
"${{ needs.build-wheel.result }}" == "failure" || \
"${{ needs.python-unit-tests.result }}" == "failure" || \
"${{ needs.unit-tests.result }}" == "failure" || \
"${{ needs.gateway-e2e.result }}" == "failure" || \
"${{ needs.go-unit-tests.result }}" == "failure" || \
"${{ needs.go-bindings-e2e.result }}" == "failure" ]]; then
echo "One or more jobs failed"
exit 1
else
Expand Down
Loading