Skip to content

fix: Relax openai and pydantic pins to match vllm requirements (#9007) [r26.10] - #9011

Merged
mc-nv merged 1 commit into
r26.10from
devin/1791586039-r26.10-vllm-compat
Oct 10, 2026
Merged

mc-nv merged 1 commit into
r26.10from
devin/1791586039-r26.10-vllm-compat

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

What does the PR do?

  • Cherry-picks fix: Relax openai and pydantic pins to match vllm requirements #9007 (20e8b513) from main to r26.10 (clean pick, -x).
  • 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:

Caveats:

None; identical diff to #9007.

Related Issues:

  • Resolves: TRI-2007

Link to Devin session: https://nvidia-cloud.devinenterprise.com/sessions/f4d32b335431447dbb84f0f9761a9110
Open in Devin Desktop: https://nvidia-cloud.devinenterprise.com/desktop/session/f4d32b335431447dbb84f0f9761a9110?variant=devin

Co-authored-by: M. Chornyi <99709299+mc-nv@users.noreply.github.com>
(cherry picked from commit 20e8b51)
@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

@mc-nv mc-nv self-assigned this Oct 9, 2026
@mc-nv
mc-nv requested review from nv-rinig, whoisj and yinggeh October 9, 2026 22:50
@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 defect was established.

Summary

Replaces exact dependency pins with ranges intended to match vLLM's requirements.

  • The OpenAI frontend can use OpenAI versions required by vLLM.
  • The Triton Python package accepts newer Pydantic 2 versions.

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

@mc-nv
mc-nv merged commit 182621b into r26.10 Oct 10, 2026
4 checks passed
@mc-nv
mc-nv deleted the devin/1791586039-r26.10-vllm-compat branch October 10, 2026 00:03
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