[https://nvbugs/5819048][fix] unwaive test of qwen3-235b eagle3

[byshiue]

Summary by CodeRabbit

• Tests
  • Re-enabled a previously skipped test by removing it from the test waives list.

Description

Test Coverage

accuracy/test_llm_api_pytorch.py::TestQwen3_235B_A22B::test_nvfp4_4gpus[latency_moe_trtllm_eagle3]

PR Checklist

Please review the following before submitting your PR:

• PR description clearly explains what and why. If using CodeRabbit's summary, please make sure it makes sense.

• PR Follows TRT-LLM CODING GUIDELINES to the best of your knowledge.

• Test cases are provided for new code paths (see test instructions)

• Any new dependencies have been scanned for license and vulnerabilities

• CODEOWNERS updated if ownership changes

• Documentation updated as needed

• Update tava architecture diagram if there is a significant design change in PR.

• The reviewers assigned automatically/manually are appropriate for the PR.

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

GitHub Bot Help

To see a list of available CI bot commands, please comment /bot help.

[byshiue]

/bot run --post-merge --stage-list "GB200-4_GPUs-PyTorch-Post-Merge-1"

[coderabbitai]

📝 Walkthrough

Walkthrough

Removed a single test skip entry from the waives list configuration file. This modification only affects test execution skipping and has no impact on production code or runtime behavior.

Changes

Cohort / File(s)	Summary
Test Skip List tests/integration/test_lists/waives.txt	Removed one SKIP entry for accuracy/test_llm_api_pytorch.py::TestQwen3_235B_A22B::test_nvfp4_4gpus[latency_moe_trtllm_eagle3], allowing this test to run instead of being skipped.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The PR description is largely empty with only the test coverage field filled; the Description section and PR title are missing.	Add a proper PR title following the template format and provide a clear description explaining the issue and why the test is being unwaived.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically describes the main change: removing a SKIP entry to unwaive a test for Qwen3-235B Eagle3, with the NVBugs issue reference provided.
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]

PR_Github #37955 [ run ] triggered by Bot. Commit: 31234c6 Link to invocation

[tensorrt-cicd]

PR_Github #37955 [ run ] completed with state SUCCESS. Commit: 31234c6
/LLM/main/L0_MergeRequest_PR pipeline #29394 (Partly Tested) completed with status: 'SUCCESS'

Link to invocation

[byshiue]

/bot skip --comment "the eagle3 test of qwen3-235B has passed"

[tensorrt-cicd]

PR_Github #37985 [ skip ] triggered by Bot. Commit: 31234c6 Link to invocation

[tensorrt-cicd]

PR_Github #37985 [ skip ] completed with state SUCCESS. Commit: 31234c6
Skipping testing for commit 31234c6

Link to invocation