Skip to content

[None][chore] Add failed cases into waives.txt#12093

Open
xinhe-nv wants to merge 3 commits intoNVIDIA:mainfrom
jieli-matrix:user/qa/post_update_waive_20260311_LLM_FUNCTION_TEST_1992
Open

[None][chore] Add failed cases into waives.txt#12093
xinhe-nv wants to merge 3 commits intoNVIDIA:mainfrom
jieli-matrix:user/qa/post_update_waive_20260311_LLM_FUNCTION_TEST_1992

Conversation

@xinhe-nv
Copy link
Collaborator

@xinhe-nv xinhe-nv commented Mar 11, 2026

waive failed cases.

Summary by CodeRabbit

  • Tests
    • Updated internal test configuration to skip specific test cases, improving test suite management and stability tracking.

Signed-off-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>
@jieli-matrix jieli-matrix marked this pull request as ready for review March 11, 2026 02:55
@jieli-matrix jieli-matrix enabled auto-merge (squash) March 11, 2026 02:55
@jieli-matrix
Copy link
Collaborator

/bot run --stage-list ""

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 11, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: e5492878-6ecc-4118-9485-329333bca396

📥 Commits

Reviewing files that changed from the base of the PR and between 2afe11d and 9103a0e.

📒 Files selected for processing (1)
  • tests/integration/test_lists/waives.txt

📝 Walkthrough

Walkthrough

Two test cases were added to the test waives list, marking them as SKIP with corresponding NV bug references.

Changes

Cohort / File(s) Summary
Test Waive Configuration
tests/integration/test_lists/waives.txt
Added two SKIP entries: test_ptp_quickstart_advanced_ngram and test_disaggregated_overlap_transceiver_runtime_python with associated NV bug identifiers.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • crazydemo
  • StanleySun639
  • LarryXFly
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is extremely minimal and lacks required sections from the template, including detailed explanation of changes, test coverage information, and the PR checklist completion. Expand the description to include: detailed explanation of which test cases are being waived and why, relevant test coverage details, and completion of the PR checklist items as appropriate.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: adding failed test cases to the waives.txt file with proper formatting.
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
  • Post copyable unit tests in a comment

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

@tensorrt-cicd
Copy link
Collaborator

PR_Github #38529 [ run ] triggered by Bot. Commit: 9103a0e Link to invocation

@tensorrt-cicd
Copy link
Collaborator

PR_Github #38529 [ run ] completed with state SUCCESS. Commit: 9103a0e
/LLM/main/L0_MergeRequest_PR pipeline #29875 (Partly Tested) completed with status: 'SUCCESS'

CI Report

Link to invocation

@jieli-matrix
Copy link
Collaborator

/bot reuse-pipeline

@tensorrt-cicd
Copy link
Collaborator

PR_Github #38576 [ reuse-pipeline ] triggered by Bot. Commit: d88bb8a Link to invocation

@tensorrt-cicd
Copy link
Collaborator

PR_Github #38576 [ reuse-pipeline ] completed with state SUCCESS. Commit: d88bb8a
Reusing PR_Github #38529 (Partly Tested) for commit d88bb8a

Link to invocation

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