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

Fix filesystem used percentage #6231

Merged

Conversation

andrewkroh
Copy link
Member

The used percentage reported by Metricbeat does not match what is reported by df. Metricbeat's computation of the system.filesystem.used.pct value was based on the total number of bytes, but df uses the total number of bytes available to the unprivileged user.

FS Blocks Used Available Use % Mount Metricbeat Used % After This Change Before This Change
/dev/vda 21137846272 16992858112 3071246336 85 / 0.846928312 0.8039067885

Fixes #5494

@andrewkroh andrewkroh added bug review help wanted Indicates that a maintainer wants help on an issue or pull request labels Jan 30, 2018
@andrewkroh andrewkroh force-pushed the bugfix/mb/sys-filesystem-used-pct branch from 346d3b0 to 83df7c2 Compare January 30, 2018 22:17
Copy link
Contributor

@ruflin ruflin left a comment

Choose a reason for hiding this comment

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

WFG

@ruflin
Copy link
Contributor

ruflin commented Jan 30, 2018

@andrewkroh What does the adopt-me label mean here? Should we backport this fix?

@andrewkroh andrewkroh added Metricbeat Metricbeat and removed help wanted Indicates that a maintainer wants help on an issue or pull request labels Jan 30, 2018
@andrewkroh
Copy link
Member Author

andrewkroh commented Jan 30, 2018

Accidentally added that label. Let's do some real world testing before back-porting it. I want to get some feedback from the users that reported the issue.

@andrewkroh andrewkroh force-pushed the bugfix/mb/sys-filesystem-used-pct branch from 83df7c2 to e9be3fb Compare January 30, 2018 23:08
@ruflin
Copy link
Contributor

ruflin commented Jan 31, 2018

@andrewkroh Sorry, needs a rebase. Changelog fun :-(

The used percentage reported by Metricbeat does not match what is reported by `df`. Metricbeat's computation of the `system.filesystem.used.pct` value was based on the total number of bytes, but df uses the total number of bytes available to the unprivileged user.

| FS       | Blocks      | Used        | Available  | Use % | Mount | Metricbeat Used % After This Change | Before This Change |
|----------|-------------|-------------|------------|-------|-------|-------------------------------------|--------------------|
| /dev/vda | 21137846272 | 16992858112 | 3071246336 | 85    | /     | 0.846928312                         | 0.8039067885       |

Fixes elastic#5494
@andrewkroh andrewkroh force-pushed the bugfix/mb/sys-filesystem-used-pct branch from e9be3fb to 51c7249 Compare January 31, 2018 16:28
@andrewkroh
Copy link
Member Author

Rebased to fix conflict.

@ruflin ruflin merged commit e605495 into elastic:master Feb 3, 2018
@andrewkroh andrewkroh deleted the bugfix/mb/sys-filesystem-used-pct branch April 20, 2018 00:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants