[https://nvbugs/5451272][fix] unwaive the test

[Shixiaowei02]

Summary by CodeRabbit

• Tests
  • Updated test suite configurations to manage test execution parameters.

[Shixiaowei02]

/bot run

[coderabbitai]

📝 Walkthrough

Walkthrough

The pull request updates the test waives configuration file by removing a skip entry for a disaggregated test and adding a skip entry for a Triton server test. The modification updates which tests are excluded from execution.

Changes

Cohort / File(s)	Summary
Test waives configuration tests/integration/test_lists/waives.txt	Removed skip entry for disaggregated/test_disaggregated.py::test_disaggregated_diff_max_tokens[TinyLlama-1.1B-Chat-v1.0] and added skip entry for triton_server/test_triton.py::test_python_bls_unit_tests[python-bls-unit-tests]

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description Check	⚠️ Warning	The pull request lacks any substantive description from the author. While the PR title appears to follow the required format ([https://nvbugs/5451272][fix] unwaive the test), the PR body contains no actual description, test coverage information, or completed checklist items. The template structure is present in the repository guidelines, but the author has not filled in any of the required sections, leaving reviewers without context about why the changes were made or what testing was performed to validate them. Based on the raw summary, the changes involve modifying waives.txt by removing and adding test entries, but this information should be clearly articulated in the PR description itself.	The author must add a proper PR description that includes: (1) an explanation of the changes and their purpose, (2) a test coverage section detailing which tests validate the changes, and (3) a completed PR checklist indicating that the coding guidelines, test coverage, dependencies, and documentation requirements have been reviewed. At minimum, the description should explain what test entries were modified in waives.txt and the rationale for these changes related to NVBugs 5451272.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The pull request title "[https://nvbugs/5451272][fix] unwaive the test" follows the required template format with a valid NVBugs ID, the [fix] type indicator, and a clear summary. The title directly aligns with the changeset, which removes one test entry and adds another in a waives.txt file, effectively representing the action of "unwaiving" tests. The title is concise, specific, and clearly communicates the primary change to someone reviewing the git history.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[tensorrt-cicd]

PR_Github #22054 [ run ] triggered by Bot. Commit: 7488dca

[tensorrt-cicd]

PR_Github #22054 [ run ] completed with state SUCCESS. Commit: 7488dca
/LLM/main/L0_MergeRequest_PR pipeline #16629 completed with status: 'SUCCESS'
Pipeline passed with automatic retried tests. Check the rerun report for details.