[None][infra] Waive 2 failed cases for main in pre-merge 34569

[ZhanruiSunCh]

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by @SimengLiu-nv)
Target branch: main
Bug(s): 6093820

Waive entries added

cpp/test_multi_gpu.py::TestDisagg::test_symmetric_executor[gpt-2proc-nixl_kvcache-90] SKIP (https://nvbugs/6093820)
cpp/test_multi_gpu.py::TestDisagg::test_symmetric_executor[gpt-2proc-ucx_kvcache-90] SKIP (https://nvbugs/6093820)

---

This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.

Summary by CodeRabbit

• Tests
  • Added test execution waivers for known issues affecting specific test configurations. These waivers skip certain test variants to improve test suite reliability while underlying issues are being addressed.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 3e08d9da-d1bb-43a5-8209-bab0f6d1b5bf

📥 Commits

Reviewing files that changed from the base of the PR and between 66431d8 and 5fbdb37.

📒 Files selected for processing (1)

• tests/integration/test_lists/waives.txt

---

📝 Walkthrough

Walkthrough

This pull request adds two new SKIP waivers to the test waiver list for a specific multi-GPU test case with different KV cache implementations. Both waivers reference the same bug tracker issue.

Changes

Cohort / File(s)	Summary
Test Waivers tests/integration/test_lists/waives.txt	Added 2 SKIP waivers for cpp/test_multi_gpu.py::TestDisagg::test_symmetric_executor[gpt-2proc-*-kvcache-90] with nixl_kvcache and ucx_kvcache variants, both linked to nvbugs/6093820.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• NVIDIA/TensorRT-LLM#12589: Also adds auto-generated SKIP waiver entries to the same test waiver file for different failing test cases.

Suggested reviewers

• chzblych

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarizes the main change: adding 2 test waivers for the main branch in pre-merge 34569, following the required format with [None][infra] prefix.
Description check	✅ Passed	The description clearly explains what was added (2 waiver entries), links to the relevant bug (6093820), and provides specific test names, though it deviates from the template structure by being auto-generated.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[SimengLiu-nv]

/bot run

[tensorrt-cicd]

PR_Github #44233 [ run ] triggered by Bot. Commit: 5fbdb37 Link to invocation

[tensorrt-cicd]

PR_Github #44233 [ run ] completed with state FAILURE. Commit: 5fbdb37
/LLM/main/L0_MergeRequest_PR pipeline #34655 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

Link to invocation