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:

  • Adds DiskHealthGet RPCs to spdk.proto
  • Returns a typed DiskHealthGetResponse whose fields align to SPDK’s bdev_nvme_get_controller_health_info

Special notes for your reviewer:

None

Additional documentation or context

None

@c3y1huang c3y1huang self-assigned this Nov 13, 2025
@c3y1huang c3y1huang force-pushed the 12016-monitor-node-disk-health branch from 0ab5c80 to a7bafb5 Compare November 13, 2025 04:22
@derekbit derekbit force-pushed the 12016-monitor-node-disk-health branch from a7bafb5 to fa3845f Compare November 13, 2025 14:26
@c3y1huang c3y1huang force-pushed the 12016-monitor-node-disk-health branch from fa3845f to 467f5ab Compare November 20, 2025 00:56
Signed-off-by: Chin-Ya Huang <chin-ya.huang@suse.com>
longhorn/longhorn-12016

Signed-off-by: Chin-Ya Huang <chin-ya.huang@suse.com>
longhorn/longhorn-12016

Signed-off-by: Chin-Ya Huang <chin-ya.huang@suse.com>
longhorn/longhorn-12016

Signed-off-by: Chin-Ya Huang <chin-ya.huang@suse.com>
longhorn/longhorn-12016

Signed-off-by: Chin-Ya Huang <chin-ya.huang@suse.com>
@c3y1huang c3y1huang force-pushed the 12016-monitor-node-disk-health branch from 467f5ab to d9950b3 Compare November 21, 2025 00:48
@c3y1huang c3y1huang marked this pull request as ready for review November 25, 2025 00:33
@c3y1huang c3y1huang requested review from a team and derekbit November 25, 2025 00:33
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 490b3f3 into longhorn:main Nov 25, 2025
2 checks passed
@c3y1huang c3y1huang deleted the 12016-monitor-node-disk-health branch November 26, 2025 00:19
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