Skip to content

[None][chore] Remove closed bugs#11527

Merged
xinhe-nv merged 1 commit intoNVIDIA:mainfrom
xinhe-nv:user/qa/unwaive_20260215
Feb 27, 2026
Merged

[None][chore] Remove closed bugs#11527
xinhe-nv merged 1 commit intoNVIDIA:mainfrom
xinhe-nv:user/qa/unwaive_20260215

Conversation

@xinhe-nv
Copy link
Collaborator

@xinhe-nv xinhe-nv commented Feb 15, 2026

Remove closed bugs from waives.txt

Summary by CodeRabbit

  • Tests
    • Updated test waiver configurations to adjust which tests are skipped during integration testing.
    • remove trt flow tests from L0

@xinhe-nv xinhe-nv marked this pull request as ready for review February 26, 2026 05:35
@xinhe-nv xinhe-nv enabled auto-merge (squash) February 26, 2026 05:35
@xinhe-nv
Copy link
Collaborator Author

/bot run

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1754dcc and a13445c.

📒 Files selected for processing (1)
  • tests/integration/test_lists/waives.txt
💤 Files with no reviewable changes (1)
  • tests/integration/test_lists/waives.txt

📝 Walkthrough

Walkthrough

Updates the test waiver list by removing 14 SKIP entries and adjusting test skip configurations, modifying which tests are excluded from execution without altering test logic or behavior.

Changes

Cohort / File(s) Summary
Test Waiver Configuration
tests/integration/test_lists/waives.txt
Removed 14 SKIP waiver entries for specific tests (e.g., accuracy/test_cli_flow.py, unittest/_torch/auto_deploy/unit/multigpu/test_ad_build_small_multi.py) and adjusted skip patterns to reconfigure which tests are excluded from execution.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

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

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description 'Remove closed bugs from waives.txt' is minimal but covers the main purpose; however, it lacks required template sections like Test Coverage and PR Checklist details. Expand the description to include test coverage information and complete the PR Checklist section to ensure the submission meets the template requirements.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title '[None][chore] Remove closed bugs' directly matches the changeset, which removes SKIP entries from the waives.txt file related to closed bugs.
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

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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

@tensorrt-cicd
Copy link
Collaborator

PR_Github #36872 [ run ] triggered by Bot. Commit: a13445c Link to invocation

@tensorrt-cicd
Copy link
Collaborator

PR_Github #36872 [ run ] completed with state FAILURE. Commit: a13445c

Link to invocation

@xinhe-nv
Copy link
Collaborator Author

/bot run

@tensorrt-cicd
Copy link
Collaborator

PR_Github #36897 [ run ] triggered by Bot. Commit: 44bc6e4 Link to invocation

@tensorrt-cicd
Copy link
Collaborator

PR_Github #36897 [ run ] completed with state SUCCESS. Commit: 44bc6e4
/LLM/main/L0_MergeRequest_PR pipeline #28568 completed with status: 'FAILURE'

⚠️ 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

@xinhe-nv
Copy link
Collaborator Author

/bot run

@tensorrt-cicd
Copy link
Collaborator

PR_Github #36918 [ run ] triggered by Bot. Commit: 7d12881 Link to invocation

@tensorrt-cicd
Copy link
Collaborator

PR_Github #36918 [ run ] completed with state SUCCESS. Commit: 7d12881
/LLM/main/L0_MergeRequest_PR pipeline #28586 completed with status: 'FAILURE'

⚠️ 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

@xinhe-nv xinhe-nv force-pushed the user/qa/unwaive_20260215 branch from 64a3a41 to 4386eb6 Compare February 26, 2026 13:37
@xinhe-nv
Copy link
Collaborator Author

/bot run

@xinhe-nv xinhe-nv requested a review from EmmaQiaoCh February 26, 2026 13:40
@tensorrt-cicd
Copy link
Collaborator

PR_Github #36938 [ run ] triggered by Bot. Commit: fb65b54 Link to invocation

@tensorrt-cicd
Copy link
Collaborator

PR_Github #36938 [ run ] completed with state FAILURE. Commit: fb65b54
/LLM/main/L0_MergeRequest_PR pipeline #28601 completed with status: 'FAILURE'

⚠️ 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

@xinhe-nv xinhe-nv force-pushed the user/qa/unwaive_20260215 branch from fb65b54 to 20a82e3 Compare February 27, 2026 01:50
@xinhe-nv
Copy link
Collaborator Author

/bot run

@tensorrt-cicd
Copy link
Collaborator

PR_Github #36997 [ run ] triggered by Bot. Commit: 20a82e3 Link to invocation

@xinhe-nv xinhe-nv force-pushed the user/qa/unwaive_20260215 branch from 20a82e3 to c5a9012 Compare February 27, 2026 02:30
@tensorrt-cicd
Copy link
Collaborator

PR_Github #36997 [ run ] completed with state SUCCESS. Commit: 20a82e3
/LLM/main/L0_MergeRequest_PR pipeline #28645 completed with status: 'SUCCESS'
Pipeline passed with automatic retried tests. Check the rerun report for details.

Link to invocation

Signed-off-by: Xin He (SW-GPU) <200704525+xinhe-nv@users.noreply.github.com>
@xinhe-nv xinhe-nv force-pushed the user/qa/unwaive_20260215 branch from c5a9012 to 61d983c Compare February 27, 2026 09:17
@xinhe-nv
Copy link
Collaborator Author

/bot reuse-pipeline

@tensorrt-cicd
Copy link
Collaborator

PR_Github #37054 [ reuse-pipeline ] triggered by Bot. Commit: 61d983c Link to invocation

@tensorrt-cicd
Copy link
Collaborator

PR_Github #37054 [ reuse-pipeline ] completed with state SUCCESS. Commit: 61d983c
Reusing PR_Github #36997 for commit 61d983c

Link to invocation

@xinhe-nv xinhe-nv merged commit ab99ddf into NVIDIA:main Feb 27, 2026
5 checks passed
@xinhe-nv xinhe-nv deleted the user/qa/unwaive_20260215 branch February 27, 2026 11:12
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Mar 9, 2026
Signed-off-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>
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