Repository navigation
[None][infra] restore L40S pre-merge stage 2 - #21241
Conversation
Signed-off-by: hanjingtian <312433810+hanjingtian@users.noreply.github.com>
|
/bot run --stage-list "L40S-PyTorch-2" |
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Remove L40S-PyTorch-2 entries jenkins/config/maintenance_stages.txt |
The configuration no longer includes the L40S-PyTorch-2 maintenance reason or its two stage entries. |
Priority:
Estimated code review effort: 1 (Trivial) | ~3 minutes
Change: Bug fix
Possibly related PRs
- NVIDIA/TensorRT-LLM#21173: Added the same L40S-PyTorch-2 maintenance reason and stage entries that this change removes.
Suggested reviewers: chzblych
Merge Risk: 
990ed
-
The maintenance pause is removed for the two L40S stage-2 entries. No actionable merge-blocking issue is identified; targeted Jenkins validation remains to be run.
Pre-merge checks: 
-
Check name Status Explanation Title check The title "[None][infra] restore L40S pre-merge stage 2" follows the repository template (ticket tag, type tag, summary). It states the main change: the L40S stage-2 maintenance pause is removed so th… Description check The description covers the summary and validation sections. It explains that L40S-PyTorch-2 and L40S-PyTorch-2-cbts are removed from maintenance_stages.txt and that stage definitions, routing, tests, … Docstring Coverage No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat… 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: Title check
-
Explanation
The title "[None][infra] restore L40S pre-merge stage 2" follows the repository template (ticket tag, type tag, summary). It states the main change: the L40S stage-2 maintenance pause is removed so the stages can run again. The raw summary confirms that L40S-PyTorch-2 and its two stage entries were removed from maintenance_stages.txt, which matches the title.
Full details: Description check
-
Explanation
The description covers the summary and validation sections. It explains that L40S-PyTorch-2 and L40S-PyTorch-2-cbts are removed from maintenance_stages.txt and that stage definitions, routing, tests, and waivers are unchanged. It lists validation steps and notes that GPU recovery is not yet verified. It does not fill in the template's Description, Test Coverage, or PR Checklist sections, and it does not state whether the change adds API changes or has a label. These gaps are non-critical for a config-only change, so the description is mostly complete.
Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files.
-
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.
|
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. |
|
PR_Github #81340 [ run ] triggered by Bot. Commit: |
|
PR_Github #81340 [ run ] completed with state |
|
/bot skip --comment "restore L40s stage" |
|
PR_Github #81367 [ skip ] triggered by Bot. Commit: |
|
PR_Github #81367 [ skip ] completed with state |
Summary
Remove the temporary L40S stage-2 maintenance pause introduced in #21173 so the affected stages can run again.
L40S-PyTorch-2andL40S-PyTorch-2-cbtsfromjenkins/config/maintenance_stages.txt.Validation
reasonempty andpatternsempty) when applying this PR diff to main.git diff --check: passed.L40S-PyTorch-2will be triggered on this PR; GPU recovery is not yet verified.Dev Engineer Review
The change removes the maintenance reason and the
L40S-PyTorch-2andL40S-PyTorch-2-cbtsentries fromjenkins/config/maintenance_stages.txt. This allows those stages to run instead of being skipped for maintenance. The H100 maintenance entries remain. No stage definitions or routing changes are reported.QA Engineer Review
No test changes.
Per-File QA Perspective
jenkins/config/maintenance_stages.txt: Verify thatL40S-PyTorch-2andL40S-PyTorch-2-cbtsare eligible to run after the maintenance entries are removed. GPU recovery and stage execution results are not established by the supplied evidence.