[None][test] Waive 3 DGX_B200 main-side CI flakes (kv-cache eviction, qwen3 fp8 lora, gemma4 e2e) - #17663
Conversation
These three tests flake on main CI independent of any change, failing across hundreds of unrelated MRs and many distinct users, with flake-typical signatures (hang/timeout, terminate-unexpectedly, assertion). Under fail-fast each flaky red gates the whole pipeline. Waive with tracking bugs: - kv_cache/test_kv_cache_v2_scheduler.py::TestKVCacheV2Llama::test_chunked_prefill_eviction_block_reuse (nvbugs/6607481) - unittest/_torch/modules/tests_lora_modules/test_qwen3_sanity.py::TestQwen3LoRA::test_qwen3_fp8_lora (nvbugs/6607487) - unittest/_torch/modeling/test_gemma4_e2e_dummy.py::test_e2e_text_31b_dummy (nvbugs/6607482) Signed-off-by: Brian Nguyen <brnguyen@nvidia.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
WalkthroughThe waiver list adds three skipped integration test cases. Each entry references an NVBugs ticket. ChangesIntegration test waivers
Estimated code review effort: 1 (Trivial) | ~2 minutes Mergeability Score: ⚪ Minimal · up to This change only waives three documented flaky CI tests and does not alter product code or runtime behavior. No actionable merge-blocking risk remains after normal review and checks. Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
/bot run |
|
PR_Github #66072 [ run ] triggered by Bot. Commit: |
|
/bot skip --comment "waive only PR" |
|
PR_Github #66082 [ skip ] triggered by Bot. Commit: |
|
PR_Github #66072 [ run ] completed with state |
|
PR_Github #66082 [ skip ] completed with state |
Description
Three unit/integration tests flake on
mainCI independent of any change under test — they fail intermittently across hundreds of unrelated MRs and many distinct users, all with flake-typical signatures (hang/timeout, terminate-unexpectedly, assertion). Under fail-fast each flaky red gates the whole pipeline, so they block unrelated PRs. Waiving them inwaives.txt, each with a filed tracking bug.kv_cache/test_kv_cache_v2_scheduler.py::TestKVCacheV2Llama::test_chunked_prefill_eviction_block_reuseunittest/_torch/modules/tests_lora_modules/test_qwen3_sanity.py::TestQwen3LoRA::test_qwen3_fp8_loraunittest/_torch/modeling/test_gemma4_e2e_dummy.py::test_e2e_text_31b_dummyEvidence from the whole-of-main stability report (trtllm-infra), 14-day window. These surfaced on the CI of #17414; that PR cherry-picks these same three waives to unblock and will rebase them away once this lands on main.
Test Coverage
Test-list-only change (
tests/integration/test_lists/waives.txt). No code paths changed; the three tests are skipped in CI until their tracking bugs are resolved.PR Checklist
Dev Engineer Review
tests/integration/test_lists/waives.txt.nvbugs/6607481nvbugs/6607487nvbugs/6607482QA Engineer Review
tests/integration/test_lists/waives.txtchanged.test-db/orqa/files were modified.