Use parse_value in the conntrack plugin and submit the result even if it's zero
authorLouis Opter <louis@dotcloud.com>
Tue, 28 Feb 2012 17:23:06 +0000 (18:23 +0100)
committerFlorian Forster <octo@collectd.org>
Wed, 29 Feb 2012 09:34:21 +0000 (10:34 +0100)
commita1a7bcd42316459361cce17e4742e2f2eb664c60
treeee392baf029baa5d1f32d9c99bb368bc3a10507c
parentd01c62d62182d1291760fbdb7ed0244991d52b5e
Use parse_value in the conntrack plugin and submit the result even if it's zero

Dear collectd,

Please find attached a patch for the conntrack plugin.

The patch is about two things:

1. submit the value even if it is zero (which is a legitimate value according
   to types.db);
2. use parse_value and a value_t instead of directly using a double and atof(3).

The first point was important because it meant that the metric was not created
when the initial value was zero. (It could also lead to holes in your graphs).

The parse_value return value is correctly checked, note that the parsed file
ends with a \n which mean that parse_value always complain when running in
debug/info maybe we should replace it with a \0 before handing the buffer to
parse_value() ?

Thanks

--
Louis Opter

Signed-off-by: Florian Forster <octo@collectd.org>
src/conntrack.c