Skip to content

Conversation

@c3y1huang
Copy link
Contributor

Which issue(s) this PR fixes:

Issue longhorn/longhorn#12016

What this PR does / why we need it:

  • Add DiskHealthGet methods to retrieve block disk health info.

Special notes for your reviewer:

None

Additional documentation or context

None

@c3y1huang c3y1huang self-assigned this Nov 14, 2025
@mergify
Copy link

mergify bot commented Nov 16, 2025

This pull request is now in conflict. Could you fix it @c3y1huang? 🙏

@c3y1huang c3y1huang force-pushed the 12016-monitor-node-disk-health branch 2 times, most recently from bc868ea to 5764796 Compare November 21, 2025 01:49
@mergify
Copy link

mergify bot commented Nov 23, 2025

This pull request is now in conflict. Could you fix it @c3y1huang? 🙏

@c3y1huang c3y1huang force-pushed the 12016-monitor-node-disk-health branch from 5764796 to cdb6401 Compare November 25, 2025 00:54
@c3y1huang c3y1huang marked this pull request as ready for review November 25, 2025 01:24
@c3y1huang c3y1huang requested review from a team and derekbit November 25, 2025 01:24
@c3y1huang c3y1huang force-pushed the 12016-monitor-node-disk-health branch 2 times, most recently from 98684b9 to 8ba0000 Compare November 27, 2025 04:32
longhorn/longhorn-12016

Signed-off-by: Chin-Ya Huang <[email protected]>
longhorn/longhorn-12016

Signed-off-by: Chin-Ya Huang <[email protected]>
longhorn/longhorn-12016

Signed-off-by: Chin-Ya Huang <[email protected]>
@c3y1huang c3y1huang force-pushed the 12016-monitor-node-disk-health branch from 8ba0000 to d23aa8b Compare November 27, 2025 07:23
Copy link
Member

@derekbit derekbit left a comment

Choose a reason for hiding this comment

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

LGTM

@derekbit derekbit merged commit ce72ce2 into longhorn:master Nov 27, 2025
12 checks passed
@c3y1huang c3y1huang deleted the 12016-monitor-node-disk-health branch November 27, 2025 08:06
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