Skip to content

[smg][ci] migrate chat completions tests to new infrastructure and build wheel once and share via artifact#16709

Merged
slin1237 merged 2 commits intomainfrom
smg-ci-n/24
Jan 8, 2026
Merged

[smg][ci] migrate chat completions tests to new infrastructure and build wheel once and share via artifact#16709
slin1237 merged 2 commits intomainfrom
smg-ci-n/24

Conversation

@slin1237
Copy link
Collaborator

@slin1237 slin1237 commented Jan 8, 2026

  • Add e2e_test/chat_completions/test_openai_server.py migrated from e2e_grpc/basic
  • Uses pytest fixtures and markers instead of unittest
  • Add chat-completions job to CI workflow
  • Split maturin-build-test into build-wheel and python-unit-tests jobs
  • build-wheel uploads wheel artifact, GPU jobs download instead of building
  • Remove rust dependencies, sccache, rust-cache from gateway-e2e jobs
  • Saves ~3 min per GPU job (~15 min total across 5 jobs)

Checklist

Review Process

  1. Ping Merge Oncalls to start the PR flow. See the PR Merge Process.
  2. Get approvals from CODEOWNERS and other reviewers.
  3. Trigger CI tests with comments (/tag-run-ci-label, /rerun-failed-ci, /tag-and-rerun-ci) or contact authorized users to do so.
  4. After green CI and required approvals, ask Merge Oncalls to merge.

- Add e2e_test/chat_completions/test_openai_server.py migrated from e2e_grpc/basic
- Uses pytest fixtures and markers instead of unittest
- Add chat-completions job to CI workflow
@gemini-code-assist
Copy link
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@slin1237 slin1237 added the run-ci label Jan 8, 2026
@slin1237 slin1237 force-pushed the smg-ci-n/24 branch 5 times, most recently from 3456684 to 2414d15 Compare January 8, 2026 14:16
- Split maturin-build-test into build-wheel and python-unit-tests jobs
- build-wheel runs on 4-gpu-a10 for fast builds (~3 min vs 11 min)
- build-wheel uploads wheel artifact, other jobs download instead of building
- Remove rust dependencies from gateway-e2e jobs
- Saves ~3 min per GPU job
@slin1237 slin1237 merged commit 2f8a363 into main Jan 8, 2026
56 of 62 checks passed
@slin1237 slin1237 deleted the smg-ci-n/24 branch January 8, 2026 14:29
hnyls2002 pushed a commit that referenced this pull request Jan 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments