Skip to content

[openapi] refactor render related openapi [3/N]#36749

Merged
vllm-bot merged 1 commit intovllm-project:mainfrom
andyxning:log_http_exception_in_handler
Mar 11, 2026
Merged

[openapi] refactor render related openapi [3/N]#36749
vllm-bot merged 1 commit intovllm-project:mainfrom
andyxning:log_http_exception_in_handler

Conversation

@andyxning
Copy link
Copy Markdown
Contributor

@andyxning andyxning commented Mar 11, 2026

Purpose

This is the third pr for #31164.

  1. refactor render related openapi to raise exception directly.
  2. refactor OpenAIServingRender's create_error_response to the one in vllm.entrypoint.utils.

Test Plan

Test Result


Essential Elements of an Effective PR Description Checklist
  • The purpose of the PR, such as "Fix some issue (link existing issues this PR will resolve)".
  • The test plan, such as providing test command.
  • The test results, such as pasting the results comparison before and after, or e2e results
  • (Optional) The necessary documentation update, such as updating supported_models.md and examples for a new model.
  • (Optional) Release notes update. If your change is user facing, please update the release notes draft in the Google Doc.

…s one

Signed-off-by: Andy Xie <andy.xning@gmail.com>
@andyxning andyxning requested a review from njhill as a code owner March 11, 2026 06:31
@andyxning andyxning changed the title [openapi] refactor render related openapi [openapi] refactor render related openapi [3/N] Mar 11, 2026
@mergify mergify bot added the frontend label Mar 11, 2026
@DarkLight1337 DarkLight1337 enabled auto-merge (squash) March 11, 2026 07:38
@github-actions github-actions bot added the ready ONLY add when PR is ready to merge/full CI is needed label Mar 11, 2026
@vllm-bot vllm-bot merged commit 40c0461 into vllm-project:main Mar 11, 2026
45 of 48 checks passed
@andyxning andyxning deleted the log_http_exception_in_handler branch March 11, 2026 10:50
wendyliu235 pushed a commit to wendyliu235/vllm-public that referenced this pull request Mar 18, 2026
Signed-off-by: Andy Xie <andy.xning@gmail.com>
fxdawnn pushed a commit to fxdawnn/vllm that referenced this pull request Mar 19, 2026
Signed-off-by: Andy Xie <andy.xning@gmail.com>
khairulkabir1661 pushed a commit to khairulkabir1661/vllm that referenced this pull request Mar 27, 2026
Signed-off-by: Andy Xie <andy.xning@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend ready ONLY add when PR is ready to merge/full CI is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants