-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
fix(inputs.zfs): Unbreak datasets stats gathering in case listsnaps is enabled on a zfs pool #12307
Conversation
Thanks so much for the pull request! |
@lytboris can you please sign the CLA otherwise we cannot review and merge your code. |
!signed-cla |
… a zfs pool Introduce explicit dataset type filter for `zfs list` command. We are interested in filesystems and volumes only.
Download PR build artifacts for linux_amd64.tar.gz, darwin_amd64.tar.gz, and windows_amd64.zip. 👍 This pull request doesn't change the Telegraf binary size 📦 Click here to get additional PR build artifactsArtifact URLs |
@lytboris thanks for this PR and signing the CLA. Can I also ask that you file an issue and explain why you are making this change and how it does not break backwards compatibility? thanks! |
Here's the issue: #12314 For backward compatibility concern: this PR is semi-fix and semi-feature: it will still work properly on ZFS pools with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the bug detailing the scenario and the fix!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Thanks for your contribution @lytboris!
Introduce explicit dataset type filter for
zfs list
command. We are interested in filesystems and volumes only.fixes: #12314