Skip to content

[TRTLLM-14727][test] Clean up MX qualification follow-ups - #17834

Merged
chienchunhung merged 1 commit into
NVIDIA:mainfrom
chienchunhung:codex/mx-e2e-followups
Aug 19, 2026
Merged

[TRTLLM-14727][test] Clean up MX qualification follow-ups#17834
chienchunhung merged 1 commit into
NVIDIA:mainfrom
chienchunhung:codex/mx-e2e-followups

Conversation

@chienchunhung

@chienchunhung chienchunhung commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Remove the unused serviceInitContainerConfig declaration and empty pod-template interpolation left behind when ModelExpress services moved from restartable init containers to regular sidecars.
  • Document TRTLLM_MX_E2E_TIMEOUT_S, including its 1200-second default and the baseline-worker, receiver-worker, and donor-readiness phases it controls.

Scope

Signed-off-by: Chien-Chun Hung <2679986+chienchunhung@users.noreply.github.com>
@chienchunhung
chienchunhung marked this pull request as ready for review August 17, 2026 18:03
@chienchunhung
chienchunhung requested review from a team as code owners August 17, 2026 18:03
@chienchunhung

Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #66822 [ run ] triggered by Bot. Commit: df74ced Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #66822 [ run ] completed with state SUCCESS. Commit: df74ced
/LLM/main/L0_MergeRequest_PR pipeline #54388 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #66896 [ run ] triggered by Bot. Commit: df74ced Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #66896 [ run ] completed with state FAILURE. Commit: df74ced
/LLM/main/L0_MergeRequest_PR pipeline #54453 completed with status: 'UNSTABLE'

CI Report

⚠️ Multi-GPU Label Required:
Multi-GPU tests require the ci: full pre-merge approved label on this PR. Ask a member of NVIDIA/trt-llm-ci-approvers to add the label, then re-trigger CI with the same bot command (no rebase needed).

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

Link to invocation

@ZhanruiSunCh ZhanruiSunCh left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM for infra part.

@chienchunhung

Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@chienchunhung
chienchunhung enabled auto-merge (squash) August 18, 2026 19:34
@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #67145 [ run ] triggered by Bot. Commit: df74ced Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #67145 [ run ] completed with state FAILURE. Commit: df74ced
/LLM/main/L0_MergeRequest_PR pipeline #54678 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@chienchunhung

Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #67193 [ run ] triggered by Bot. Commit: df74ced Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #67193 [ run ] completed with state SUCCESS. Commit: df74ced
/LLM/main/L0_MergeRequest_PR pipeline #54719 completed with status: 'SUCCESS'

CI Report

Link to invocation

@chienchunhung
chienchunhung merged commit 2419a0f into NVIDIA:main Aug 19, 2026
17 checks passed
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.

4 participants