Skip to content

[https://nvbugs/6210714][test] Unwaive TestQwen3_5_35B_A3B fp8 block reuse test - #16997

Merged
VALLIS-NERIA merged 1 commit into
NVIDIA:mainfrom
VALLIS-NERIA:user/xiweny/unwaive-6210714
Jul 29, 2026
Merged

[https://nvbugs/6210714][test] Unwaive TestQwen3_5_35B_A3B fp8 block reuse test#16997
VALLIS-NERIA merged 1 commit into
NVIDIA:mainfrom
VALLIS-NERIA:user/xiweny/unwaive-6210714

Conversation

@VALLIS-NERIA

@VALLIS-NERIA VALLIS-NERIA commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

Description

Unwaive accuracy/test_llm_api_pytorch.py::TestQwen3_5_35B_A3B::test_fp8[enable_block_reuse=True].

The issue tracked by https://nvbugspro.nvidia.com/bug/6210714 is no longer reproducible, so the corresponding SKIP entry is removed from tests/integration/test_lists/waives.txt.

Test Coverage

The unwaived test itself provides the coverage; it now runs in CI again.

PR Checklist

  • PR title follows [JIRA/NVBUG/None][type] description format
  • Commit is signed off (DCO)

🤖 Generated with Claude Code

Dev Engineer Review

  • Removed the single waiver for TestQwen3_5_35B_A3B::test_fp8[enable_block_reuse=True] associated with NVBug 6210714.
  • The change is correctly scoped to tests/integration/test_lists/waives.txt, with no code or public API changes.
  • Test-list format and entry removal are consistent; no duplicate or unintended entries were introduced.

QA Engineer Review

  • No test-db/ or qa/ files were modified, and no test functions were changed.
  • The previously waived test will run in CI again.
  • Verdict: needs follow-up pending CBTS coverage data.

…reuse test (By Agent)

Signed-off-by: Xiwen Yu <13230610+VALLIS-NERIA@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: e0d3c7d6-d5ed-459a-8d3f-b0d72a9c6e46

📥 Commits

Reviewing files that changed from the base of the PR and between 9d508eb and 2670a92.

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

Walkthrough

The integration waiver list removes the waiver for the Qwen3.5 35B A3B FP8 test case with block reuse enabled.

Changes

Test waiver update

Layer / File(s) Summary
Remove Qwen3.5 FP8 waiver
tests/integration/test_lists/waives.txt
Deletes the waiver entry for TestQwen3_5_35B_A3B::test_fp8[enable_block_reuse=True].

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested reviewers: bowenfu

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title is specific and accurately summarizes the waiver removal for the affected test case.
Description check ✅ Passed The description follows the template with Description, Test Coverage, and PR Checklist sections filled in.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@VALLIS-NERIA

Copy link
Copy Markdown
Collaborator Author

/bot run

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #62458 [ run ] triggered by Bot. Commit: 2670a92 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #62458 [ run ] completed with state SUCCESS. Commit: 2670a92
/LLM/main/L0_MergeRequest_PR pipeline #50611 completed with status: 'SUCCESS'

CI Report

Link to invocation

@VALLIS-NERIA
VALLIS-NERIA merged commit 2f113fd into NVIDIA:main Jul 29, 2026
12 checks passed
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