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

[Metricbeat] system.cpu.total.*.pct includes IOWait #7627

Closed
axw opened this issue Jul 18, 2018 · 2 comments
Closed

[Metricbeat] system.cpu.total.*.pct includes IOWait #7627

axw opened this issue Jul 18, 2018 · 2 comments
Labels
libbeat Metricbeat Metricbeat module

Comments

@axw
Copy link
Member

axw commented Jul 18, 2018

The total CPU% is calculated as the amount of time spent in the non-idle state:

beats/libbeat/metric/system/cpu/cpu.go

Line 119 in 4946202

return common.Round(float64(numCPU)-calculatePct(s0.Idle, s1.Idle), common.DefaultDecimalPlacesCount)

I would expect that IOWait time to be excluded also.
@ruflin
Copy link
Contributor

ruflin commented Jul 18, 2018

Note: This also effects the data reported to Monitoring.

@tsg tsg mentioned this issue Jul 24, 2018
Merged
tsg added a commit to tsg/beats that referenced this issue Jul 24, 2018
@tsg
Exclude IOWait time from the CPU total
7653ac5 
This affects the following two metrics:

* `system.cpu.total.pct`
* `system.cpu.total.norm.pct`

Fixes elastic#7627.
ruflin pushed a commit that referenced this issue Jul 24, 2018
@tsg @ruflin
Exclude IOWait time from the CPU total (#7691)
dd5938e 
This affects the following two metrics:

* `system.cpu.total.pct`
* `system.cpu.total.norm.pct`

Fixes #7627.
@henrikno
Copy link
Contributor

I was curious why this changed, but then I read http://veithen.github.io/2013/11/18/iowait-linux.html which explained it pretty well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
libbeat Metricbeat Metricbeat module
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@jsoriano @henrikno @ruflin @axw