Skip to content

[None][test] Waive 4 failed cases for main in QA test list - #14233

Merged
crazydemo merged 1 commit into
NVIDIA:mainfrom
xinhe-nv:trtllm-ci-report/waive-20260518-023838
May 19, 2026
Merged

[None][test] Waive 4 failed cases for main in QA test list#14233
crazydemo merged 1 commit into
NVIDIA:mainfrom
xinhe-nv:trtllm-ci-report/waive-20260518-023838

Conversation

@xinhe-nv

@xinhe-nv xinhe-nv commented May 18, 2026

Copy link
Copy Markdown
Collaborator

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by qa@nvidia.com)
Target branch: main
Bug(s): 6185173, 6185182

Waive entries added

accuracy/test_llm_api_pytorch.py::TestQwen3_5_4B::test_dflash SKIP (https://nvbugs/6185182)
accuracy/test_llm_api_autodeploy.py::TestNemotronV2::test_fp8[True] SKIP (https://nvbugs/6185173)
accuracy/test_llm_api_autodeploy.py::TestNemotronV2::test_auto_dtype[True] SKIP (https://nvbugs/6185173)
accuracy/test_llm_api_autodeploy.py::TestNemotronV2::test_auto_dtype[False] SKIP (https://nvbugs/6185173)

This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.

Summary by CodeRabbit

  • Tests
    • Added test skip waivers for specific model accuracy test cases related to auto-dtype and flash attention configurations, referencing tracked issues for resolution.

Review Change Stack

@crazydemo
crazydemo force-pushed the trtllm-ci-report/waive-20260518-023838 branch from 39bca7a to 5e10c68 Compare May 18, 2026 02:41
@coderabbitai

coderabbitai Bot commented May 18, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

This pull request adds four new test skip waiver entries to the test skip list. Three waivers are added for NemotronV2 autodeploy accuracy tests, and one waiver is added for a Qwen3_5_4B pytorch accuracy test, each linked to their respective nvbugs.

Changes

Test Waivers

Layer / File(s) Summary
Add accuracy test waivers
tests/integration/test_lists/waives.txt
Insert three waivers for accuracy/test_llm_api_autodeploy.py::TestNemotronV2 covering test_auto_dtype[False], test_auto_dtype[True], and test_fp8[True] (nvbug 6185173), and one waiver for accuracy/test_llm_api_pytorch.py::TestQwen3_5_4B::test_dflash (nvbug 6185182).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • NVIDIA/TensorRT-LLM#14086: Adds similar SKIP waiver entries to the same tests/integration/test_lists/waives.txt file for failing accuracy tests.
  • NVIDIA/TensorRT-LLM#14136: Modifies the same waiver file by adding SKIP entries for accuracy/test_llm_api_pytorch.py test failures.
  • NVIDIA/TensorRT-LLM#13984: Updates the same waiver file with SKIP entries for NemotronV2 and Qwen-related accuracy test cases.

Suggested reviewers

  • crazydemo
  • jieli-matrix
  • StanleySun639
  • LarryXFly
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description check ✅ Passed The description includes the bug references, lists all waive entries with their associated NVBugs IDs, and clearly states it is auto-generated. However, it lacks the structured sections from the template (Description, Test Coverage, PR Checklist).
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.
Title check ✅ Passed The title accurately describes the main change: adding waivers for 4 failed test cases in the QA test list.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

Bug(s): 6185173, 6185182
Requested by: qa@nvidia.com

Signed-off-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>
@crazydemo
crazydemo force-pushed the trtllm-ci-report/waive-20260518-023838 branch from 5e10c68 to af4591e Compare May 18, 2026 02:43
@crazydemo
crazydemo enabled auto-merge (squash) May 18, 2026 02:44
@crazydemo crazydemo changed the title [None][infra] Waive 4 failed cases for main in post-merge [None][test] Waive 4 failed cases for main in QA test list May 18, 2026
@crazydemo

Copy link
Copy Markdown
Collaborator

/bot run --stage-list ""

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #48812 [ run ] triggered by Bot. Commit: af4591e Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #48812 [ run ] completed with state SUCCESS. Commit: af4591e
/LLM/main/L0_MergeRequest_PR pipeline #38574 (Partly Tested) completed with status: 'SUCCESS'

CI Report

Link to invocation

@crazydemo

Copy link
Copy Markdown
Collaborator

/bot --reuse-pipeline

@github-actions

Copy link
Copy Markdown

GitHub Bot Help

/bot [-h] ['run', 'kill', 'skip', 'reuse-pipeline'] ...

Provide a user friendly way for developers to interact with a Jenkins server.

Run /bot [-h|--help] to print this help message.

See details below for each supported subcommand.

Details

run [--reuse-test (optional)pipeline-id --disable-fail-fast --skip-test --stage-list "A10-PyTorch-1, xxx" --gpu-type "A30, H100_PCIe" --test-backend "pytorch, cpp" --add-multi-gpu-test --only-multi-gpu-test --disable-multi-gpu-test --post-merge --extra-stage "H100_PCIe-TensorRT-Post-Merge-1, xxx" --detailed-log --debug(experimental) --high-priority]

Launch build/test pipelines. All previously running jobs will be killed.

--reuse-test (optional)pipeline-id (OPTIONAL) : Allow the new pipeline to reuse build artifacts and skip successful test stages from a specified pipeline or the last pipeline if no pipeline-id is indicated. If the Git commit ID has changed, this option will be always ignored. The DEFAULT behavior of the bot is to reuse build artifacts and successful test results from the last pipeline.

--disable-reuse-test (OPTIONAL) : Explicitly prevent the pipeline from reusing build artifacts and skipping successful test stages from a previous pipeline. Ensure that all builds and tests are run regardless of previous successes.

--disable-fail-fast (OPTIONAL) : Disable fail fast on build/tests/infra failures.

--skip-test (OPTIONAL) : Skip all test stages, but still run build stages, package stages and sanity check stages. Note: Does NOT update GitHub check status.

--stage-list "A10-PyTorch-1, xxx" (OPTIONAL) : Only run the specified test stages. Supports wildcard * for pattern matching (e.g., "*PerfSanity*" matches all stages containing PerfSanity). Examples: "A10-PyTorch-1, xxx", "PerfSanity". Note: Does NOT update GitHub check status.

--gpu-type "A30, H100_PCIe" (OPTIONAL) : Only run the test stages on the specified GPU types. Examples: "A30, H100_PCIe". Note: Does NOT update GitHub check status.

--test-backend "pytorch, cpp" (OPTIONAL) : Skip test stages which don't match the specified backends. Only support [pytorch, cpp, tensorrt, triton]. Examples: "pytorch, cpp" (does not run test stages with tensorrt or triton backend). Note: Does NOT update GitHub pipeline status.

--only-multi-gpu-test (OPTIONAL) : Only run the multi-GPU tests. Note: Does NOT update GitHub check status.

--disable-multi-gpu-test (OPTIONAL) : Disable the multi-GPU tests. Note: Does NOT update GitHub check status.

--add-multi-gpu-test (OPTIONAL) : Force run the multi-GPU tests in addition to running L0 pre-merge pipeline.

--post-merge (OPTIONAL) : Run the L0 post-merge pipeline instead of the ordinary L0 pre-merge pipeline.

--extra-stage "H100_PCIe-TensorRT-Post-Merge-1, xxx" (OPTIONAL) : Run the ordinary L0 pre-merge pipeline and specified test stages. Supports wildcard * for pattern matching. Examples: --extra-stage "H100_PCIe-TensorRT-Post-Merge-1, xxx", --extra-stage "Post-Merge".

--detailed-log (OPTIONAL) : Enable flushing out all logs to the Jenkins console. This will significantly increase the log volume and may slow down the job.

--debug (OPTIONAL) : Experimental feature. Enable access to the CI container for debugging purpose. Note: Specify exactly one stage in the stage-list parameter to access the appropriate container environment. Note: Does NOT update GitHub check status.

--high-priority (OPTIONAL) : Run the pipeline with high priority. This option is restricted to authorized users only and will route the job to a high-priority queue.

kill

kill

Kill all running builds associated with pull request.

skip

skip --comment COMMENT

Skip testing for latest commit on pull request. --comment "Reason for skipping build/test" is required. IMPORTANT NOTE: This is dangerous since lack of user care and validation can cause top of tree to break.

reuse-pipeline

reuse-pipeline

Reuse a previous pipeline to validate current commit. This action will also kill all currently running builds associated with the pull request. IMPORTANT NOTE: This is dangerous since lack of user care and validation can cause top of tree to break.

@crazydemo

Copy link
Copy Markdown
Collaborator

/bot reuse-pipeline

1 similar comment
@crazydemo

Copy link
Copy Markdown
Collaborator

/bot reuse-pipeline

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

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

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #48872 [ reuse-pipeline ] completed with state SUCCESS. Commit: af4591e
Reusing PR_Github #48812 (Partly Tested) for commit af4591e

Link to invocation

@crazydemo
crazydemo disabled auto-merge May 19, 2026 02:49
@crazydemo
crazydemo merged commit e483bf8 into NVIDIA:main May 19, 2026
11 checks passed
KleinBlueC pushed a commit to KleinBlueC/TensorRT-LLM that referenced this pull request May 19, 2026
)

Signed-off-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>
@xinhe-nv
xinhe-nv deleted the trtllm-ci-report/waive-20260518-023838 branch May 27, 2026 13:31
bmarimuthu-nv pushed a commit to nv-auto-deploy/TensorRT-LLM that referenced this pull request May 28, 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.

4 participants