Skip to content

Added stat for disks in "replacing" state#737

Closed
Finomosec wants to merge 14 commits into
prometheus:masterfrom
Finomosec:master
Closed

Added stat for disks in "replacing" state#737
Finomosec wants to merge 14 commits into
prometheus:masterfrom
Finomosec:master

Conversation

@Finomosec

Copy link
Copy Markdown

When a disk is replaced using the command mdadm /dev/md0 --replace /dev/sdb1 --with /dev/sde1 the disk ist marked with (R).

This information is now exported as count DisksReplacing.

@Finomosec

Copy link
Copy Markdown
Author

I fixed the problems preventing a merge. Please merge it now, so the connected PR prometheus/node_exporter#3007 can be merged too.

@reloaded

Copy link
Copy Markdown

Really, no one has merged in this PR? I need to monitor my mdadm array with Prometheus + Grafana, but I'm in need of exactly what @Finomosec set out to fix for themself and our community. Is there any intention on merging his PRs so we can monitor the mdadm arrays when they're undergoing recovery/repair?

He has another PR in the prometheus/node_exporter repo that fixes some issues that are holding me up from finishing my mdadm array monitoring.

prometheus/node_exporter#3007

@discordianfish

Copy link
Copy Markdown
Member

The DCO sign-off is missing

@SuperQ

SuperQ commented Oct 25, 2025

Copy link
Copy Markdown
Member

This needs a rebase.

@SuperQ

SuperQ commented Oct 25, 2025

Copy link
Copy Markdown
Member

I think this is solved by MDStatComponent in #674.

@discordianfish

Copy link
Copy Markdown
Member

Lets close for now then

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.

4 participants