Skip to content

fix:remove duplicated trust_remote_code knob from trtllm-serve - #5143

Merged
nv-guomingz merged 1 commit into
NVIDIA:mainfrom
nv-guomingz:user/guomingz/clean_duplilcate_serve_option
Jun 12, 2025
Merged

fix:remove duplicated trust_remote_code knob from trtllm-serve#5143
nv-guomingz merged 1 commit into
NVIDIA:mainfrom
nv-guomingz:user/guomingz/clean_duplilcate_serve_option

Conversation

@nv-guomingz

@nv-guomingz nv-guomingz commented Jun 12, 2025

Copy link
Copy Markdown
Collaborator

Remove the duplicated trust_remote_code knob which raised by #4799

Comment thread tensorrt_llm/commands/serve.py Outdated
@nv-guomingz
nv-guomingz force-pushed the user/guomingz/clean_duplilcate_serve_option branch from 8b6cb0b to f72d6f6 Compare June 12, 2025 03:04
@nv-guomingz

Copy link
Copy Markdown
Collaborator Author

/bot run

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #8589 [ run ] triggered by Bot

@yechank-nvidia

Copy link
Copy Markdown
Collaborator

Hi @nv-guomingz, I already created PR here.

@nv-guomingz

Copy link
Copy Markdown
Collaborator Author

Hi @nv-guomingz, I already created PR here.

No worries, let's use #5072 for removing this duplicate knob.

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #8589 [ run ] completed with state SUCCESS
/LLM/main/L0_MergeRequest_PR pipeline #6232 completed with status: 'SUCCESS'

@nv-guomingz nv-guomingz reopened this Jun 12, 2025
Signed-off-by: nv-guomingz <137257613+nv-guomingz@users.noreply.github.com>
@nv-guomingz
nv-guomingz force-pushed the user/guomingz/clean_duplilcate_serve_option branch from f72d6f6 to 3708031 Compare June 12, 2025 11:35
@nv-guomingz

Copy link
Copy Markdown
Collaborator Author

/bot reuse-pipeline

@nv-guomingz nv-guomingz self-assigned this Jun 12, 2025
@nv-guomingz
nv-guomingz enabled auto-merge (squash) June 12, 2025 11:37
@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #8673 [ reuse-pipeline ] triggered by Bot

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #8673 [ reuse-pipeline ] completed with state SUCCESS
Reusing PR_Github #8589 for commit 3708031

@nv-guomingz
nv-guomingz merged commit 58d4ca2 into NVIDIA:main Jun 12, 2025
@nv-guomingz
nv-guomingz deleted the user/guomingz/clean_duplilcate_serve_option branch September 30, 2025 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants