Repository navigation
[None][ci] waive pre-existing test failures on main - #20735
Conversation
…s on main; tracking bugs in the PR description) Signed-off-by: Brian Nguyen <brnguyen@nvidia.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
WalkthroughThe integration test waiver list adds ChangesTest waiver
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~3 minutes Change: Other Suggested reviewers: Merge Risk: ⚪ Minimal · up to
|
| Check name | Status | Explanation |
|---|---|---|
| Title check |  |
The title clearly identifies a CI waiver for pre-existing test failures on main. It follows the repository format with a None ticket prefix and a concise summary. |
| Description check | |
The description explains the pre-existing failures, tracking bug, blocked pull request, waiver purpose, and pr-babysitter behavior. It provides sufficient context for this waiver-only change, although… |
| Docstring Coverage | |
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… |
| Linked Issues check | |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
Full details: Description check
-
Explanation
The description explains the pre-existing failures, tracking bug, blocked pull request, waiver purpose, and pr-babysitter behavior. It provides sufficient context for this waiver-only change, although it does not include the full standard section headings or checklist.
Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Comment @coderabbitai help to get the list of available commands.
|
/bot run |
|
PR_Github #79581 [ run ] triggered by Bot. Commit: |
|
PR_Github #79581 [ run ] completed with state
|
|
Automatically added "ci: full pre-merge approved" because this PR has satisfied the required GitHub review approvals. Unresolved review conversations and other required checks remain independent merge requirements. |
|
/bot skip --comment "waive only PR" |
|
PR_Github #79673 [ skip ] triggered by Bot. Commit: |
|
PR_Github #79673 [ skip ] completed with state |
Waives for pre-existing test failures on
main, each with a filed tracking bug. These failures block unrelated PRs; landing the waives here keeps them out of those PRs' diffs.unittest/_torch/thop/parallelMaintained by pr-babysitter: entries are pruned automatically as
mainpicks their tests up, and the PR closes itself when nothing is left to carry. The branch is regenerated on each change — do not push to it.Dev Engineer Review
The change adds a waiver for
unittest/_torch/thop/parallelintests/integration/test_lists/waives.txt, linked to NVBug 6966263. It changes only test gating and introduces no source-code or public API changes. Review findings are unavailable from the supplied evidence.QA Engineer Review
The change modifies only the waiver list. It suppresses known failures for
unittest/_torch/thop/parallel; no test implementation or test ID changed. No test result is supplied. Coverage verdict: needs follow-up because the waiver can hide ongoing failures.Per-File QA Perspective
tests/integration/test_lists/waives.txt: Adds theunittest/_torch/thop/parallelwaiver for NVBug 6966263. The entry is a test-gating change and does not require a CI or manual-QA test-list update.