ensure collectd doesn't OOM if unable to connect to graphite

RESOLVED FIXED

Status

RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: dividehex, Unassigned)

Tracking

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
This hasn't happened yet but we should definitely enable Queue limits in collectd just in case. (I thought it was already implemented but default but I was wrong)

https://github.com/collectd/collectd/pull/280
(Reporter)

Comment 1

5 years ago
Created attachment 832455 [details] [diff] [review]
bug938739-1.patch

This patch adds a hard limit of 6000 metrics.  This should keep about 30 mins of metrics. (~200 metrics polled per min * 30mins = 6000)

I haven't looked at the metric struct to estimate a metrics footprint size but can't imaging it being more than few hundred bytes.

http://collectd.org/documentation/manpages/collectd.conf.5.shtml#global_options
Attachment #832455 - Flags: review?(dustin)
Attachment #832455 - Flags: review?(dustin) → review+
(Reporter)

Updated

5 years ago
Attachment #832455 - Flags: checked-in+
(Reporter)

Updated

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