Skip to content

fix(rack-controller): recover power-blocked firmware updates [backport 2.1] - #5068

Merged
nv-dmendoza merged 3 commits into
NVIDIA:release/v2.1from
kunzhao-nv:fix/v2.1-rack-fw-off-target
Aug 17, 2026
Merged

fix(rack-controller): recover power-blocked firmware updates [backport 2.1]#5068
nv-dmendoza merged 3 commits into
NVIDIA:release/v2.1from
kunzhao-nv:fix/v2.1-rack-fw-off-target

Conversation

@kunzhao-nv

@kunzhao-nv kunzhao-nv commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Backports #5031 to release/v2.1.

Rack firmware maintenance can leave a rack permanently stuck in Maintenance(FirmwareUpgrade(WaitForComplete)) when a scoped machine is Ready with desired_power_state == Off. The rack writes host_reprovisioning_requested, but the machine power-manager gate prevents the request from being consumed, so the rack controller waits forever and later maintenance requests remain blocked.

This backport rejects that condition before RMS submission and recovers racks already stuck by it by marking the firmware job failed, clearing maintenance_requested and credentials, conditionally clearing unconsumed rack-owned host requests, and transitioning the rack to Error. It also includes the accepted review follow-ups from #5031: one canonical rack-maintenance initiator formatter, non-panicking persisted-job handling, and removal of a shadowed binding. Pending device IDs are included in wait outcomes for diagnosis.

Related issues

Type of Change

  • Add - New feature or capability
  • Change - Changes in existing functionality
  • Fix - Bug fixes
  • Remove - Removed features or deprecated functionality
  • Internal - Internal changes (refactoring, tests, docs, etc.)

Breaking Changes

  • This PR contains breaking changes

Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing performed

Additional Notes

Signed-off-by: Kun Zhao <kunzhao@nvidia.com>
(cherry picked from commit cf642eb)
Signed-off-by: Kun Zhao <kunzhao@nvidia.com>
Signed-off-by: Kun Zhao <kunzhao@nvidia.com>
(cherry picked from commit d464f8b)
Signed-off-by: Kun Zhao <kunzhao@nvidia.com>
Signed-off-by: Kun Zhao <kunzhao@nvidia.com>
@kunzhao-nv
kunzhao-nv requested a review from a team August 17, 2026 19:12
@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: ac678c26-ce9e-445b-bbdd-68ef898a6f67

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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

@github-actions

Copy link
Copy Markdown

🔐 TruffleHog Secret Scan

No secrets or credentials found!

Your code has been scanned for 700+ types of secrets and credentials. All clear! 🎉

🔗 View scan details

🕐 Last updated: 2026-08-17 19:14:37 UTC | Commit: 551f738

@nv-dmendoza nv-dmendoza left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Backports needed fix

@nv-dmendoza
nv-dmendoza merged commit 44d0245 into NVIDIA:release/v2.1 Aug 17, 2026
123 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