stop using the collectd aggregation module

RESOLVED FIXED

Status

Infrastructure & Operations
RelOps: Puppet
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: dividehex, Assigned: dividehex)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
Cpu metrics aggregation can be done in the cpu module itself now.  This will workaround the 'double up' bug we experienced with the aggregation module.

https://github.com/collectd/collectd/issues/297
(Assignee)

Comment 1

3 years ago
Created attachment 8622804 [details] [diff] [review]
bug1173636-1.patch

Configuration change to collectd cpu modules; sums up all cpu cores and cpu active states into a single percentage.  This is what was being attempted via the aggregation module and doesn't require the graphite derivative func to be applied to convert jiffies to percentage.

This will also start a new metric name for all hosts under cpu.percent.active
Attachment #8622804 - Flags: review?(arich)
(Assignee)

Comment 2

3 years ago
Comment on attachment 8622804 [details] [diff] [review]
bug1173636-1.patch

Also adds cpu module back to osx since it was busted in previous versions with a temperature read value error.  Once this lands we can remove the associated papertrail filter
Attachment #8622804 - Flags: review?(arich) → review+
(Assignee)

Comment 3

3 years ago
Comment on attachment 8622804 [details] [diff] [review]
bug1173636-1.patch

remote:   https://hg.mozilla.org/build/puppet/rev/3e3358cf38c3
remote:   https://hg.mozilla.org/build/puppet/rev/eebc9068348f
Attachment #8622804 - Flags: checked-in+
(Assignee)

Updated

3 years ago
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.