Skip to content

fix: Relax openai and pydantic pins to match vllm requirements - #9007

Merged
mc-nv merged 1 commit into
mainfrom
devin/tri-2007-vllm-compat-fixes
Oct 9, 2026
Merged

mc-nv merged 1 commit into
mainfrom
devin/tri-2007-vllm-compat-fixes

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor
CI (internal): [#72738572](http://tritonserver.local/ci/pipelines/72738572)

What does the PR do?

  • Cherry-picks 6dc3ff1e from mchornyi/TRI-1959/1.0dev to main.
  • python/openai/requirements.txt: openai==1.107.3 -> openai>=2.25.0,<3.
  • src/python/pyproject.toml: pydantic==2.10.6 -> pydantic>=2.12.0,<3.
  • Exact pins downgraded both packages in the vllm image, so tool_parsers failed to import NamespaceTool and the vllm model could not load.

Checklist

  • PR title reflects the change and is of format <commit_type>: <Title>
  • Changes are described in the pull request.
  • Related issues are referenced.
  • Populated github labels field
  • Added test plan and verified test passes.
  • Verified that the PR passes existing CI.
  • Verified copyright is correct on all changed files.
  • Added succinct git squash message before merging ref.
  • All template sections are filled out.

Commit Type:

  • fix

Related PRs:

Where should the reviewer start?

python/openai/requirements.txt, src/python/pyproject.toml

Test plan:

  • vLLM L0 jobs pass on the default branch.

  • CI Pipeline ID: 72738572

Caveats:

Only 6dc3ff1e is ported; the 26.09 release-line commits are not.

Related Issues:

  • Resolves: TRI-2007

vllm requires openai>=2.25.0 and pydantic>=2.12.0. The exact pins
openai==1.107.3 and pydantic==2.10.6 downgraded both in the vllm image,
so vllm's tool_parsers failed to import NamespaceTool and the vllm model
could not load.
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@greptile-apps

greptile-apps Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[High impact] The PR appears safe to merge; no actionable issue was found.

Summary

This PR replaces two exact dependency pins with ranges that match vLLM's stated requirements.

  • The Python packages can use dependency versions vLLM requires.

Reviews (1) · Last reviewed commit: "fix: Relax openai and pydantic pins to m..." · Reviewed by Greptile

@mc-nv
mc-nv merged commit 20e8b51 into main Oct 9, 2026
4 checks passed
@mc-nv
mc-nv deleted the devin/tri-2007-vllm-compat-fixes branch October 9, 2026 22:45
mc-nv added a commit that referenced this pull request Oct 10, 2026
… [r26.10] (#9011)

Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: M. Chornyi <99709299+mc-nv@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants