[trtllm-agent]

Description

Temporarily waive 1 failed case(s) on DGX_H100. Opened automatically by auto-waive (draft; a human reviews and merges).

The native main-break detector (native-p3/k3-w12h-fresh12h-nopass-hostclock/1.0.1) reports disaggregated/test_disaggregated.py::test_disaggregated_context_first_bf16_ownership broken on main for DGX_H100: failure signature runtimeerror: # worker(s) exited during startup: /path (rc=#), onset 2026-11-05T04:51:24Z, latest failure 2026-11-05T15:11:01Z. Policy native_p3@1.0.0 selected it; https://nvbugs/7051902 tracks the fix. Scope DGX_H100 is the stage prefix that ran the failing test.

• Bug: https://nvbugs/7051902 (details, logs and 7-day test history are in the bug)
• Restore coverage when: Remove this line when https://nvbugs/7051902 is fixed or closed and the test passes on DGX_H100 post-merge; un-waive follows the bug, not the PR.
• Scope reviewed by: auto-waive at 2026-11-05T16:05:49.045376+00:00

Tests in this break event:

• disaggregated/test_disaggregated.py::test_disaggregated_context_first_bf16_ownership on DGX_H100

PRs whose pre-merge runs hit this break (rebase or re-run after this merges):

• [TRTLLM-17005][refactor] Component-local C++ Tests (Part 5) #21024
• [https://nvbugs/6988867][fix] Bind the disagg test server to a kernel-assigned port #21249
• [TRTLLM-17527][refactor] transport typed Python request inputs #21375
• [https://nvbugs/6770978][fix] Allow cold FlashInfer JIT in B300 Mamba disagg startup #21591

full:DGX_H100/disaggregated/test_disaggregated.py::test_disaggregated_context_first_bf16_ownership SKIP (https://nvbugs/7051902)

What to check before merging:

• The bug's test-history links show the cases failing on main, not only on the trigger PR.
• The scope DGX_H100 matches the failing stage and no narrower waiver already exists.
• Nothing else is needed from you after merging: the bug owner removes these lines with the fix.

Provenance

• Detector native-p3/k3-w12h-fresh12h-nopass-hostclock/1.0.1, policy native_p3@1.0.0
• Run b2679f67-9e1c-407e-a303-e337949777f9; snapshot SHA-256 bf238dc37a1a674c5920684005308a7cfad7e1619c5a98e0bcd30b6d049c4109; config SHA-256 09cf0ee078a35cefbb2080f3d321976a2a18bc5d6e6c6b3de9d637db1c92730b
• disaggregated/test_disaggregated.py::test_disaggregated_context_first_bf16_ownership: incident 56dc2aaef48f7e991ea73fdf1f7bc58747d9c2f8150d33576171428b4d322727, observation 52a0254ac0d92e812907a61a60a09743caa9982e5ea7671f5e5c83a51a4d6960

Test Coverage

Review the incident evidence and selector scope before merging. This proposal does not
establish that main is fixed. Confirm restoration separately after the corrective change.

PR Checklist

• Please check this after reviewing the above items as appropriate for this PR.

Dev Engineer Review

tests/integration/test_lists/waives.txt adds three DGX_H100 skip entries associated with NVBug 7051902. The entries cover one disaggregated integration test and two unit tests. This is a test-selection change; it does not fix the underlying behavior. Remove the waivers after the bug is fixed and the tests pass.

QA Engineer Review

The waiver list adds these test IDs: test_disaggregated_context_first_bf16_ownership, test_context_first_bf16_owned_transfer, and test_context_first_bf16_lifecycle_gpu. All three entries use the full:DGX_H100 selector and reference NVBug 7051902. The change skips these tests; it does not provide test results or verify a fix. Coverage verdict: needs follow-up.

Per-File QA Perspective

• tests/integration/test_lists/waives.txt: Adds three DGX_H100 waivers for disaggregated ownership, KV transfer, and transfer lifecycle tests. Confirm the waivers are removed after the bug is fixed and the tests pass.

[tburt-nv]

/bot skip --comment "waive only"

[github-actions]

Automatically added "ci: full pre-merge approved" because this PR has satisfied the required GitHub review approvals. Unresolved review conversations and other required checks remain independent merge requirements.

[coderabbitai]

No actionable comments were generated in the recent review.

Walkthrough

Three DGX_H100 context-first BF16 tests were added to the integration skip list. Each entry references NVBug 7051902. The existing allreduce-strategy accuracy test remains skipped.

Changes

Integration test waivers

Layer / File(s)	Summary
Add DGX_H100 test waivers tests/integration/test_lists/waives.txt	The skip list now includes the context-first BF16 disaggregated ownership, owned-transfer, and GPU transfer-lifecycle tests. Each entry references NVBug 7051902.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Suggested reviewers: lori-ren

Merge Risk: Minimal · up to fe45d

Three DGX_H100 tests will be temporarily skipped; no actionable merge-blocking code defect was established.

Pre-merge checks: 5

Check name	Status	Explanation
Title check		The title identifies the DGX_H100 waiver, but says one case was waived; the change summary reports three added tests.
Description check		The description includes the required sections and explains the waiver. It describes one test, but the change summary reports three added tests. Test Coverage does not identify completed validation.
Docstring Coverage		No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches

Generate unit tests (beta)

• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Recent review info

Run configuration

• Source mode: Auto
• Repository: NVIDIA/TensorRT-LLM
• Branch: main
• File: .coderabbit.yaml
• Commit: 7a06809
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Enterprise
• Run ID: 34f91fb0-1f69-4fcf-8e54-fe05c1af9f7b

Commits

Reviewing files that changed from the base of the PR and between 7a06809 and fe45d5e.

Files selected for processing: 1

• tests/integration/test_lists/waives.txt

Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 11 remain after this review.

[tensorrt-cicd]

PR_Github #83120 [ skip ] triggered by Bot. Commit: fe45d5e Link to invocation

[tensorrt-cicd]

PR_Github #83120 [ skip ] completed with state SUCCESS. Commit: fe45d5e
Skipping testing for commit fe45d5e

Link to invocation