Skip to content

sysfs: add NVMeSubsystemClass for NVMe-oF subsystem parsing#797

Open
sradco wants to merge 1 commit intoprometheus:masterfrom
sradco:add-nvmesubsystem-sysfs-parsing
Open

sysfs: add NVMeSubsystemClass for NVMe-oF subsystem parsing#797
sradco wants to merge 1 commit intoprometheus:masterfrom
sradco:add-nvmesubsystem-sysfs-parsing

Conversation

@sradco
Copy link

@sradco sradco commented Mar 16, 2026

Add NVMeSubsystemClass() method to sysfs.FS that reads
/sys/class/nvme-subsystem/ to discover NVMe over Fabrics subsystems
and their controller paths. For each subsystem it reads the NQN,
model, serial, I/O policy, and enumerates controllers with their
state, transport, and address.

This complements the existing NVMeClass() (which reads per-controller
hardware info from /sys/class/nvme/) by providing subsystem-level
path redundancy data. All read errors are propagated to the caller.

This is the parsing layer for prometheus/node_exporter#3579

Signed-off-by: Shirly Radco sradco@redhat.com
Co-authored-by: AI Assistant noreply@cursor.com

Add NVMeSubsystemClass() method to sysfs.FS that reads
/sys/class/nvme-subsystem/ to discover NVMe over Fabrics subsystems
and their controller paths. For each subsystem it reads the NQN,
model, serial, I/O policy, and enumerates controllers with their
state, transport, and address.

This complements the existing NVMeClass() (which reads per-controller
hardware info from /sys/class/nvme/) by providing subsystem-level
path redundancy data. All read errors are propagated to the caller.

Signed-off-by: Shirly Radco <sradco@redhat.com>
Co-authored-by: AI Assistant <noreply@cursor.com>
@sradco
Copy link
Author

sradco commented Mar 18, 2026

Hi @SuperQ @jsafrane , Please review this PR. Thank you.

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.

1 participant