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.

Sign up for GitHub

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

Jump to bottom

util: make inode metrics optional in FilesystemNodeGetVolumeStats() #3407

Merged
merged 1 commit into from
Oct 13, 2022
Merged

util: make inode metrics optional in FilesystemNodeGetVolumeStats() #3407

merged 1 commit into from
Oct 13, 2022

Commits on Oct 13, 2022

  1. util: make inode metrics optional in FilesystemNodeGetVolumeStats() 

    CephFS does not have a concept of "free inodes", inodes get allocated
on-demand in the filesystem.

This confuses alerting managers that expect a (high) number of free
inodes, and warnings get produced if the number of free inodes is not
high enough. This causes alerts to always get reported for CephFS.

To prevent the false-positive alerts from happening, the
NodeGetVolumeStats procedure for CephFS (and CephNFS) will not contain
inodes in the reply anymore.

See-also: https://bugzilla.redhat.com/2128263
Signed-off-by: Niels de Vos <ndevos@redhat.com>
    @nixpanic @mergify
    nixpanic authored and mergify[bot] committed Oct 13, 2022
    Configuration menu
    Copy the full SHA
    60bcc6a View commit details
    Browse the repository at this point in the history