Add pressure stall information #126
Comments
|
Related: prometheus/node_exporter#1174 You're probably looking to implement something similar to one of the other handlers like /proc/net/dev. /cc @rtreffer |
|
Ah yes, thanks, found it! I'm basing most of my patch on the other collectors so it's going to look a bit different from what @rtreffer did in their procfs patch. Though if they want to send in theirs instead I'd be happy not to lift a finger 😉. I'm going to skip cgroups2 support for a sec since I don't have a way to test that currently, but should look at adding that later. |
remijouannet
pushed a commit
to remijouannet/procfs
that referenced
this issue
Oct 20, 2022
Closes prometheus#126 Signed-off-by: Daniele Sluijters <daenney@users.noreply.github.com>
bobrik
pushed a commit
to bobrik/procfs
that referenced
this issue
Jan 14, 2023
The functions in the `net` module all read from the `/proc/net` symlink, which always points to the current process. These new functions read from `/proc/pid/net` Closes prometheus#126 Supersedes prometheus#218
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I'd like to add support for
/proc/pressure/*that the Linux kernel exposes (to in turn add support for that to the node exporter).I started looking at it but I'm stumbling on a thing; currently
Procand itspathhelper always assumes that what we want is/proc/<pid>/<path...>however what I need access to is/proc/pressure/{cpu,memory,io}and I can't seem to find a good way to do that.Did I miss something in the
ProcAPI or should I just read the contents of those three files without leveraging theProcstruct and its methods?The text was updated successfully, but these errors were encountered: