Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[REQUEST] zpool stats monitoring method change. #383

Closed
reefland opened this issue Jul 3, 2022 · 2 comments · Fixed by #385
Closed

[REQUEST] zpool stats monitoring method change. #383

reefland opened this issue Jul 3, 2022 · 2 comments · Fixed by #385
Assignees
Labels
enhancement New feature or request

Comments

@reefland
Copy link

reefland commented Jul 3, 2022

Per #374 which is monitoring ZFS file system via /proc/spl/kstat/zfs/*pool_name*/io this will not work for long term monitoring. This has already been removed from current builds of ZFS:

$ zfs version
zfs-2.1.5-1
zfs-kmod-2.1.5-1

$ ls -l /proc/spl/kstat/zfs/vault
total 0
-rw-r--r-- 1 root root 0 Jun 30 17:40 dmu_tx_assign
-rw-r--r-- 1 root root 0 Jun 30 17:40 iostats
-rw-r--r-- 1 root root 0 Jun 30 17:40 multihost
-rw-r--r-- 1 root root 0 Jun 30 17:40 objset-0x183
-rw-r--r-- 1 root root 0 Jun 30 17:40 objset-0x195b1
-rw-r--r-- 1 root root 0 Jun 30 17:40 objset-0x36
-rw-r--r-- 1 root root 0 Jun 30 17:40 objset-0x3699
-rw-r--r-- 1 root root 0 Jun 30 17:40 objset-0x86
-rw------- 1 root root 0 Jun 30 17:40 reads
-rw-r--r-- 1 root root 0 Jun 30 17:40 state
-rw-r--r-- 1 root root 0 Jun 30 17:40 txgs

These links appear relevant to its removal:
openzfs/zfs#12212
openzfs/zfs@371f88d

Prometheus Node Exporter for ZFS was using this as well:
prometheus/node_exporter#2068

@reefland reefland added the enhancement New feature or request label Jul 3, 2022
@aristocratos
Copy link
Owner

@simplepad You wanna take a look at this?

@simplepad
Copy link
Contributor

Yes, I'm working on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants