Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

dns plugin: Check for "struct ip6_ext".

Solaris appears to declare the struct, but only when The Right defines
are specified at compile time. As a quick and dirty fix, only build with
IPv6 support when the struct is usable.

Fixes: #348

smart: KNF

Update smart.c

Are you sure this doesn't memory-leak?
(I didn't have time to run valgrind but is it possible that this command is missing?)

smart: KNF

Merge pull request #1076 from schuellerf/master

Update smart.c

Update smart.c

Are you sure this doesn't memory-leak?
(I didn't have time to run valgrind but is it possible that this command is missing?)

vmem plugin: fix types for nr_dirtied and nr_written

tcpconns plugin: Fix memory leak.

"prev" was set to NULL and never updated, leading to "port_list_head" to
be set to an entry later in the list. This leaks memory because earlier
entries in the list are now unreachable.

Fixes: #1074

tail plugin: initialize user_data_t

src/configfile.c: Set a plugin context when autoloading plugins.

Fixes: #1069

src/daemon/configfile.c: Remove unnecessary "continue" statements.

Merge remote-tracking branch 'github/pr/1067'

plugin: not return -1 if plugin_flush_callback_name fails

Do not call plugin_unregister if the read registration fails.

plugin: fix white space.

plugin: remove cast type information from malloc.

plugin: move the code to create flush/name to plugin_flush_callback_name

apcups plugin: Implement the "PersistentConnection" option.

If set to false, close the connection after each interval. This lets
users chose this method if persistent connections pose a problem.

Fixes: #617

apcups plugin: Handle "END APC" lines correctly and stop reading.

Fixes: #617

apcups plugin: Fight code rot.

* Use the cleaner "complex" config and the cf_util_* functions.
* Rename "host" and "port" to "node" and "service".
  Use cf_util_get_service() so users may specify services as string.
* Remove unused defines.

Support for call the flush callback at regular intervals using
the read plugin callback.

varnish: update plugin_register_complex_read() to new default value

... just a follow up to cce136946b, in which one of the two
plugin_register_complex_read() calls got missed out.

processes: revert re-introduction of ps_read_tasks

This commit partially revert 30eeeee996124de666f907877f8196e5580101ed
which solved incorrectly the merge conflict

Signed-off-by: Arthur Gautier <baloo@gandi.net>

src/daemon/plugin.[ch]: Use cdtime_t for the interval argument of "register complex read".

All plugins converted to "struct timespec", just so that
plugin_register_complex_read() would convert back to cdtime_t again. This patch
removed this crazyness; the new "determine automatically" value is zero (used
to be NULL).

Merge remote-tracking branch 'github/pr/1036'

Conflicts:
src/processes.c

Merge branch 'pr/1033'

processes plugin: Read number of threads from /proc/$pid/stat as a fallback.

Should reading the number of threads from /prod/$pid/status fail, use the
number provided by /prod/$pid/stat as a fallback. If that fails, too, use 1.

As discussed with @manuelluis in #1033.

statsd plugin: Dispatch NaN when no timer event was recorded.

This makes timing events appear faster when writing to RRD files.

Fixes: #1038, #1049

Fix VERSION->PACKAGE_VERSION and PACKAGE->PACKAGE_NAME so they're always defined

Fix header condition

The dependency is either UTMPX or UTMP or Statgrab. The inclusion above hasn't statgrab.h in the elif-chain, that means compilation fails when only libstatgrab is available. The line can safely be taken out as the same condition is checked in #L120.

Remove old systemd unit

collectd.service superseded by systemd.collectd.service

Merge remote-tracking branch 'github/pr/1056'

tail_csv plugin: Print warning and continue when metric is not found.

Previously, when multiple metrics were specified on one "Collect" line and one
didn't exist, all *following* metrics would be ignored. This patch changes this
to continue in this case and configure as many metrics as possible.

Plugin network: add support for the timeout argument in the flush callback

Plugin tail_csv: the manual says that you can use multiple times the
Collect option but the code expecs t"Collect n1 n2 n3 n4", fix
to allow to put the option Collect multiple times.

Plugin tail_csv: if TimeFrom is not specified in config file
the default value for time_from is -1 and the type of fileds_num
is size_t, so the comparsion of signed and unsigned not work as espected

processes: change type to contextswitch and types to derive_t

processes: use cf_util_get_boolean for get "CollectContextSwitch"

processes: delete added white spaces

processes: rename ps_ctx_switch to report_ctx_switch

processes: remove function ps_read_tasks

Processes: fix types in ps_read_status, change to unsigned long.

Processes: in ps_read_status fix check to skip lines
if it doesn't start with "Vm" and it doesn't start "Threads".

stats plugin: Fix reference of undeclared symbol.

Issue: #997

statsd plugin: Free latency counter and AVL trees.

latency counters (used by TIMER metrics) and AVL trees (used by SET metrics)
were not freed when cleaning up unused metrics. This resulted in leaked memory.

Fixes: #997

cpu plugin: Correct number of expected CPU states on Mac OS X.

This appears to be a regression introduced in f14ab935. Thanks to Robert Viduya
for reporting the bug.

Signed-off-by: Florian Forster <octo@collectd.org>

zone: add mentions to README and AUTHORS files

RPM specfile: handle new zone plugin

Merge remote-tracking branch 'origin/collectd-5.3' into collectd-5.4

Merge remote-tracking branch 'github/pr/1054'

Make flow more obvious

Add myself

No busy looping around open()

Use sizeof() instead of duplicating the variable length

Handle case when zone_scandir() returns NULL

Return -1 in zone_read on error

Get pagesize only once during initialization

Handle case when zone_find_stats() returns NULL

Use ZONENAME_MAX and handle errors returned by getzonenamebyid()

Prefix all warnings and error messages with plugin name

Use (void *) in c_avl_create()

Use type 'percent' instead of 'zone_cpu'

update changelog + bump release dates

Build system: Fix linking with libheap.la and depend on testing.h.

collectd-tg tried to link with libheap.a (instead of ….la), which
doesn't exist. testing.h was not mentioned by any target, leading to it
being missing form the tarballs.

Build system: Build tested units as libraries.

This simplifies the build rules for the tests, aka. check programs.

* test_foo.c have been renamed to foo_test.c.
* foo_test.c now reside right next to foo.c and foo.h.
* Build and refer to .la files, rather than depending on .c files from
  other directories.

Fixes: #1042

Filter chains helper: minor cleanups

This is a couple of trivial followups to 809a8973.

write_redis: Replaced method for checking for a NULL value for the redis connection

write_redis: Log error message from redis command failures

write_redis: improve checking the redis connection

Filterchain: if we don't find a writer, output the available writers to syslog.

bindings/java/Makefile.am: Add "uninstall-local" target.

Otherwise "make distcheck" will complain about leftover files.

bindings/java/Makefile.am: Use $(srcdir) when specifying dependencies.

This fixes out-of-tree builds, like "make distcheck" does.

bindings/java/Makefile.am: Add "uninstall-local" target.

Otherwise "make distcheck" will complain about leftover files.

bindings/java/Makefile.am: Use $(srcdir) when specifying dependencies.

This fixes out-of-tree builds, like "make distcheck" does.

ChangeLog: Update date to 2015-05-22.

write_sensu, write_tsdb plugins: Use GAUGE_FORMAT.

Issue #1039

Merge branch 'collectd-5.4' into collectd-5.5

Conflicts:
src/write_redis.c

write_redis: Fix timestamp sent to Redis.

Various plugins: Use the global GAUGE_FORMAT.

This fixes the postgresql, rrdtool, target_notification, write_graphite and
write_redis plugins to use the globally defined format for gauges.

Issue #1039

src/collectd.h: Define GAUGE_FORMAT.

This format is also used in format_values(), so exported gauges follow a common
format.

See also issue #1039

src/utils_format_json.[ch]: Export gauges with 15 decimal places of precision.

Fixes: #1039

ChangeLog: Import changes from the wiki.

RPM specfile: fix changelog formatting

RPM specfile: also bump release number

RPM specfile: update changelog for 5.5

Plugin process: collect voluntary  and involuntary context switches.

contrib/wiki2changelog.pl: Add script converting from wiki to ChangeLog.

ChangeLog: Change text width to 80 characters.

Bump version to 5.5.0; Update ChangeLog.

ipc plugin: Group functions by operating system.

Plugin process: in Linux read the number of threads from
/proc/pid/status instead of counter the number of files in
/proc/pid/task/*.

Merge branch 'collectd-5.4'

Merge branch 'collectd-5.3' into collectd-5.4

Merge branch 'collectd-4.10' into collectd-5.3

curl_xml: fix 3 small memory leaks

Discovered while testing the previous 2 commits. NB: valgrind
already complained about these before 0afea606 was applied, so this
isn't related to issue #935.

Fix test for tables and empty instance

Change-Id: I9a3fc3ded9566bd68c80e5fbce2d8b717ea1848d

Revert "curl_xml plugin: Fixed tautological pointer comparison error."
Fixes #931
This reverts commit 0afea60611f115a28b8ec331aba610e3038c1ef2.

Enable the processes plugin by default on Solaris.

It now builds and works correctly in both 32-bit and 64-bit mode.