Skip to content

fix(admin): show disabled heal runtime state#256

Merged
overtrue merged 2 commits into
mainfrom
overtrue/heal-status-display
Jul 13, 2026
Merged

fix(admin): show disabled heal runtime state#256
overtrue merged 2 commits into
mainfrom
overtrue/heal-status-display

Conversation

@overtrue

Copy link
Copy Markdown
Contributor

Summary

Expose the RustFS heal runtime state in admin status responses and render disabled or uninitialized states explicitly in rc admin heal status.

Changes

  • Add an optional state field to HealStatus and background heal response decoding.
  • Preserve legacy behavior when older servers omit state.
  • Render Disabled, Uninitialized, Idle, and In Progress from the explicit runtime state.

Verification

  • cargo fmt --all --check
  • cargo test -p rc-core test_heal_status_default -- --nocapture
  • cargo test -p rc-s3 test_background_heal_status_response_maps_to_heal_status -- --nocapture
  • cargo test -p rustfs-cli test_heal_status_indicator_reports_disabled_runtime -- --nocapture

Copilot AI review requested due to automatic review settings July 12, 2026 23:45

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@overtrue overtrue merged commit 7783efe into main Jul 13, 2026
16 checks passed
@overtrue overtrue deleted the overtrue/heal-status-display branch July 13, 2026 00:44
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.

2 participants