collector: add nvmesubsystem collector for NVMe-oF path health

[sradco]

Summary

Add a new disabled-by-default nvmesubsystem collector that reads
/sys/class/nvme-subsystem/ to expose NVMe over Fabrics subsystem
path health metrics.

Exposed metrics

Metric	Type	Labels	Description
node_nvmesubsystem_info	Gauge (info)	subsystem, nqn, model, serial, iopolicy	Subsystem identity
node_nvmesubsystem_namespace_info	Gauge (info)	subsystem, device	Maps namespace block device to subsystem
node_nvmesubsystem_paths	Gauge	subsystem	Total controller paths
node_nvmesubsystem_paths_live	Gauge	subsystem	Controller paths in live state
node_nvmesubsystem_path_state	Gauge	subsystem, controller, transport, state	Per-controller state

The namespace_info metric enables precise correlation between NVMe
block devices (e.g. nvme0n1) and their parent subsystems, which is
needed for workload-aware storage path health alerting in Kubernetes
environments.

Enable

--collector.nvmesubsystem

No special permissions required — reads only world-readable sysfs attributes.

Dependencies

• Stacked on build(deps): bump prometheus/procfs to v0.21.1 #3715 (build(deps): bump prometheus/procfs to v0.21.1)
• sysfs: add NVMeSubsystemClass for NVMe-oF subsystem parsing procfs#797

Test plan

• Unit tests with testutil.GatherAndCompare for all metrics
• Tests for no-devices case
• State normalization tests (normalizeControllerState)
• End-to-end test passing

[sradco]

Hi @SuperQ , I created this PR for a new multipath collector.
I would appreciate your review.

[jsafrane]

The code looks good to me. I tested it with 2 NVMe over TCP devices, I got:

node_nvmesubsystem_info{iopolicy="numa",model="Linux",nqn="tempdisk",serial="bab529a0f32e397e1319",subsystem="nvme-subsys0"} 1
node_nvmesubsystem_path_state{controller="nvme0",state="connecting",subsystem="nvme-subsys0",transport="tcp"} 0
node_nvmesubsystem_path_state{controller="nvme0",state="dead",subsystem="nvme-subsys0",transport="tcp"} 0
node_nvmesubsystem_path_state{controller="nvme0",state="live",subsystem="nvme-subsys0",transport="tcp"} 1
node_nvmesubsystem_path_state{controller="nvme0",state="resetting",subsystem="nvme-subsys0",transport="tcp"} 0
node_nvmesubsystem_path_state{controller="nvme0",state="unknown",subsystem="nvme-subsys0",transport="tcp"} 0
node_nvmesubsystem_path_state{controller="nvme1",state="connecting",subsystem="nvme-subsys0",transport="tcp"} 0
node_nvmesubsystem_path_state{controller="nvme1",state="dead",subsystem="nvme-subsys0",transport="tcp"} 0
node_nvmesubsystem_path_state{controller="nvme1",state="live",subsystem="nvme-subsys0",transport="tcp"} 1
node_nvmesubsystem_path_state{controller="nvme1",state="resetting",subsystem="nvme-subsys0",transport="tcp"} 0
node_nvmesubsystem_path_state{controller="nvme1",state="unknown",subsystem="nvme-subsys0",transport="tcp"} 0
node_nvmesubsystem_paths_live{subsystem="nvme-subsys0"} 2
node_nvmesubsystem_paths_total{subsystem="nvme-subsys0"} 2

Which looks reasonable.

[sradco]

PR is failing since its based on prometheus/procfs#797, which is still not merged

[sradco]

Hi @SuperQ , Please review this PR, Thank you in advance.

[sradco]

This PR depends on #3715 (build(deps): bump prometheus/procfs to v0.21.1).
CI will fail until #3715 is merged — the collector commit itself is ready for review.

[sradco]

Rebased

[SuperQ]

Thanks!