[None][test] Waive 4 failed cases for main in QA CI - #15505
Conversation
Signed-off-by: GitLab CI Bot <gitlab-ci@nvidia.com>
Bug(s): 5970614, 6331421, 6344107, 6344612 Requested by: qa@nvidia.com Jenkins build: https://prod.blsm.nvidia.com/swqa-tensorrt-qa-test/job/LLM_FUNCTION_CLUSTER_TEST/1552/ Signed-off-by: tensorrt-cicd <90828364+tensorrt-cicd@users.noreply.github.com> Signed-off-by: GitLab CI Bot <gitlab-ci@nvidia.com>
2a0991f to
a43adc4
Compare
|
/bot --skip comments "only waive tests" |
GitHub Bot Help
Provide a user friendly way for developers to interact with a Jenkins server. Run See details below for each supported subcommand. Details
Launch build/test pipelines. All previously running jobs will be killed.
kill
Kill all running builds associated with pull request. skip
Skip testing for latest commit on pull request. 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. |
|
/bot skip --comment "Only waive tests" |
📝 WalkthroughWalkthroughThree new ChangesB200 Integration Test Waivers
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@tests/integration/test_lists/waives.txt`:
- Around line 175-181: The TestQwen2_5_VL_7B test_auto_dtype entry on line 180
in waives.txt does not match the PR objectives. The line currently references
bug 6316983 but the PR objectives specify it should reference bug 6331421.
Either update line 180 to change the bug number from 6316983 to 6331421 to align
with the stated PR objectives, or provide clarification that this discrepancy
between the objectives and actual changes is intentional. Ensure all four
promised test waivers from the PR objectives are correctly represented in the
final diff.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: 2126718f-a522-4382-893a-b03b47e3755d
📒 Files selected for processing (1)
tests/integration/test_lists/waives.txt
|
PR_Github #55155 [ skip ] triggered by Bot. Commit: |
|
PR_Github #55155 [ skip ] completed with state |
Signed-off-by: GitLab CI Bot <gitlab-ci@nvidia.com> Signed-off-by: tensorrt-cicd <90828364+tensorrt-cicd@users.noreply.github.com> Co-authored-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com> Signed-off-by: GitLab CI Bot <gitlab-ci@nvidia.com>
Signed-off-by: GitLab CI Bot <gitlab-ci@nvidia.com> Signed-off-by: tensorrt-cicd <90828364+tensorrt-cicd@users.noreply.github.com> Co-authored-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>
Auto-generated Waive PR
Created by: TensorRT LLM CI Report (requested by qa@nvidia.com)
Target branch:
mainJenkins build: https://prod.blsm.nvidia.com/swqa-tensorrt-qa-test/job/LLM_FUNCTION_CLUSTER_TEST/1552/
Bug(s): 5970614, 6331421, 6344107, 6344612
Waive entries added
This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.
Summary by CodeRabbit