Skip to content

openai_http: document extras in BackendArgs and CLI help for backend_…#646

Open
arturofredes wants to merge 1 commit into
vllm-project:mainfrom
arturofredes:fix/openai-http-backend-extras-support
Open

openai_http: document extras in BackendArgs and CLI help for backend_…#646
arturofredes wants to merge 1 commit into
vllm-project:mainfrom
arturofredes:fix/openai-http-backend-extras-support

Conversation

@arturofredes

@arturofredes arturofredes commented Mar 19, 2026

Copy link
Copy Markdown

…kwargs

  • Add optional extras field to OpenAIHttpBackendArgs with description for merging request body fields (e.g. temperature, top_p) via backend_kwargs.
  • Update --backend-kwargs help to mention extras.body example for sampling params.
  • Backend already supported extras; this improves discoverability and schema docs.

Made-with: Cursor

Summary

Details

  • [ ]

Test Plan

Related Issues

  • Resolves #

  • "I certify that all code in this PR is my own, except as noted below."

Use of AI

  • Includes AI-assisted code completion
  • Includes code generated by an AI application
  • Includes AI-generated tests (NOTE: AI written tests should have a docstring that includes ## WRITTEN BY AI ##)

…kwargs

- Add optional extras field to OpenAIHttpBackendArgs with description for
  merging request body fields (e.g. temperature, top_p) via backend_kwargs.
- Update --backend-kwargs help to mention extras.body example for sampling params.
- Backend already supported extras; this improves discoverability and schema docs.

Made-with: Cursor
@jaredoconnell

Copy link
Copy Markdown
Collaborator

Does this help with the CLI at all?
Please refactor to fix the quality errors to pass CI.

@mergify

mergify Bot commented May 30, 2026

Copy link
Copy Markdown
Contributor

This pull request has merge conflicts that must be resolved before it can be
merged. Please rebase the PR, @arturofredes.

https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork

@mergify mergify Bot added the needs-rebase label May 30, 2026
@mergify

mergify Bot commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Hi @arturofredes, the DCO check has failed. Please click on DCO in the Checks section for instructions on how to resolve this.

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.

2 participants