Skip to content

Conversation

@ShashwatHiregoudar
Copy link

Improvement of the NVMe collector in node-exporter, to include the below-mentioned NVMe namespace-specific parameters NUSE
NCAP
NSZE
LBA Size
ANA State

Signed-off-by: Shashwat Hiregoudar <shashwathiregoudar@gmail.com>
Signed-off-by: Shashwat Hiregoudar <shashwathiregoudar@gmail.com>
@ShashwatHiregoudar
Copy link
Author

cc:
@SuperQ
@discordianfish

@discordianfish
Copy link
Member

Could you try to add tests for the new metrics?

Signed-off-by: Shashwat Hiregoudar <shashwathiregoudar@gmail.com>
Signed-off-by: Shashwat Hiregoudar <shashwathiregoudar@gmail.com>
Signed-off-by: Shashwat Hiregoudar <shashwathiregoudar@gmail.com>
@ShashwatHiregoudar
Copy link
Author

I have updated the fixtures and the output.txt.
@discordianfish @SuperQ
Please suggest additional edits if needed.

@discordianfish
Copy link
Member

Seems reasonable but parsing should go into the prometheus/procfs module

@ShashwatHiregoudar
Copy link
Author

Ok 👍

prometheus/procfs#765
This PR should do it.

please approve it

@ShashwatHiregoudar
Copy link
Author

ShashwatHiregoudar commented Nov 20, 2025

After they are merged, i can apply this patch and update the go.mod file with the new release tag
procfs.patch

@ShashwatHiregoudar
Copy link
Author

hi @discordianfish
can you please ask Ben Kochie to approve the above mentioned procfs-PR.

It's been a while.

@discordianfish discordianfish requested a review from SuperQ January 16, 2026 10:37
Copy link
Member

@discordianfish discordianfish left a comment

Choose a reason for hiding this comment

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

Oops, sorry - right still needs prometheus/procfs#765 first

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