Skip to content

fix(trtllm): log deprecation notices so operators see them (cherry-pick #11694 to release/1.3.0)#11715

Merged
nv-nmailhot merged 1 commit into
release/1.3.0from
tanmayv/cherry-pick-11694-release-1.3.0
Jul 15, 2026
Merged

fix(trtllm): log deprecation notices so operators see them (cherry-pick #11694 to release/1.3.0)#11715
nv-nmailhot merged 1 commit into
release/1.3.0from
tanmayv/cherry-pick-11694-release-1.3.0

Conversation

@tanmayv25

@tanmayv25 tanmayv25 commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Summary

Cherry-pick of #11694 (f9a67cb60b7760f3c812241d1b9bbba20c0a5e16) onto release/1.3.0.

TensorRT-LLM deprecation notices are now logged so operators actually see them. Updates components/src/dynamo/trtllm/args.py and adds coverage in test_trtllm_unit.py.

Validation

🤖 Generated with Claude Code


Open in Devin Review

Signed-off-by: tanmayv25 <tanmay2592@gmail.com>
Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
(cherry picked from commit f9a67cb)
@tanmayv25 tanmayv25 requested review from a team as code owners July 15, 2026 17:01
@copy-pr-bot

copy-pr-bot Bot commented Jul 15, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@github-actions github-actions Bot added fix backend::trtllm Relates to the trtllm backend labels Jul 15, 2026

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@nv-nmailhot nv-nmailhot merged commit d244885 into release/1.3.0 Jul 15, 2026
21 of 22 checks passed
@nv-nmailhot nv-nmailhot deleted the tanmayv/cherry-pick-11694-release-1.3.0 branch July 15, 2026 23:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend::trtllm Relates to the trtllm backend fix size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants