Skip to content

Xyne 123 bench runs#167

Merged
prakhar-prakash-juspay merged 2 commits into
release/v1.83.3from
xyne-123-bench-runs
May 27, 2026
Merged

Xyne 123 bench runs#167
prakhar-prakash-juspay merged 2 commits into
release/v1.83.3from
xyne-123-bench-runs

Conversation

@prakhar-prakash-juspay

Copy link
Copy Markdown
Collaborator

Relevant issues

Pre-Submission checklist

Please complete all items before asking a LiteLLM maintainer to review your PR

  • I have Added testing in the tests/litellm/ directory, Adding at least 1 test is a hard requirement - see details
  • My PR passes all unit tests on make test-unit
  • My PR's scope is as isolated as possible, it only solves 1 specific problem

CI (LiteLLM team)

CI status guideline:

  • 50-55 passing tests: main is stable with minor issues.
  • 45-49 passing tests: acceptable but needs attention
  • <= 40 passing tests: unstable; be careful with your merges and assess the risk.
  • Branch creation CI run
    Link:

  • CI run for the last commit
    Link:

  • Merge / cherry-pick CI run
    Links:

Type

🆕 New Feature
🐛 Bug Fix
🧹 Refactoring
📖 Documentation
🚄 Infrastructure
✅ Test

Changes

Admin-only registry of vLLM/SGLang benchmark runs, consumed by the
grid-ai-onboarding "Bench Runs" dashboard tab.

- New LiteLLM_BenchRun table: schema.prisma (root, proxy, proxy-extras) plus
  a migration (20260527000000_add_bench_run_table).
- bench_run_endpoints.py: POST /bench/run/new, GET /bench/run/list,
  POST /bench/run/delete (PROXY_ADMIN, mirrors playground_endpoints).
- Register bench_run_router in proxy_server.py.
Mirror Dockerfile.non_root: when PROXY_EXTRAS_SOURCE=local, build
litellm-proxy-extras from the fork source and replace the PyPI wheel in
/wheels, so the fork's schema.prisma + migrations ship. Unlike non_root,
the database image includes the prisma schema-engine, so `prisma migrate
deploy` runs at startup -- letting the local docker-compose setup
self-apply custom migrations (e.g. LiteLLM_BenchRun) without a separate
migrations job. Defaults to "published" so existing builds are unaffected.
Copilot AI review requested due to automatic review settings May 27, 2026 10:52

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@prakhar-prakash-juspay prakhar-prakash-juspay changed the base branch from main to release/v1.83.3 May 27, 2026 10:53
@prakhar-prakash-juspay prakhar-prakash-juspay merged commit 787236d into release/v1.83.3 May 27, 2026
43 of 58 checks passed
@codecov-commenter

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants