Bugfix for libcollectdclient lcc_putval.

Hello,

I'm using collectd to monitor system and process parameters of our code under
test running on a cluster. The test processes communicate to collectd via the
unixsock plugin, but there was a small error in the client library which I had
to fix first.

The wire format of the putval command is missing a space behind the closing
double quote of the identifier. As I said, it is a small fix, but nevertheless
I attached the patch for client.c. The bug is present in 4.6 (that is the
version I'm using in production), but I fixed it against 4.10.0.

Cheers,
Johan

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

snmp plugin: csnmp_value_list_to_value: Include the OID in the log message.

snmp plugin: Don't print a warning if the ASN type is "ASN_NULL".

Thanks to "Flyinvap" for pointing this out.

Merge remote branch 'origin/collectd-4.9' into collectd-4.9

configure.in: htonll check depends on linker

Fixing a cross-compile issue by depending only on a linker check.

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

python: Fixed some memory leaks in the write and notification callbacks.
Thanks to Volkmar Uhlig for the hint.

python: Fixed a crash if the plugin was loaded but not configured.

Merge branch 'collectd-4.8' into collectd-4.9

rrdcached.c needs to include rrd.h

Build system: Check for "strtok_r" availability with "-Wall -Werror" when using GCC.

Previously, the check was done without "-Werror" which resulted in a
warning and success. When the daemon was later built with "-Werror" set,
that warning would result in an error and abort the build.

Thanks to Aurélien Reynaud for reporting the problem.

Merge branch 'collectd-4.8' into collectd-4.9

configure.in: Put conditionals around <inttypes.h>, too, for consistency's sake.

Solaris 8 fixes: Check for <stdint.h> before inclusion.

Solaris 8 fixes: Fall back to putenv(3).

collectd-nagios.c: Output performance data with the "%f" format string.

Apparently the parser used by nagios is dumb as bread. Thanks to Lao Wai
for pointing this out. For more information, see:
<http://nagiosplug.sourceforge.net/developer-guidelines.html#AEN201>

Bumped version to 4.9.2; Updated ChangeLog.

Merge branch 'collectd-4.8' into collectd-4.9

Conflicts:
ChangeLog

Bumped version to 4.8.5; Updated ChangeLog.

Merge branch 'collectd-4.8' into collectd-4.9

Conflicts:
ChangeLog

src/plugin.c: plugin_log: Append newline to message …

… even when no log-plugin was loaded.

network plugin: Don't access values in an already freed array.

contrib/exec-nagios.px: Whitespace and POD fixes.

Bumped version to 4.8.2; Updated ChangeLog.

Merge branch 'collectd-4.8' into collectd-4.9

network plugin: Fix a severe memory leak.

A couple of bytes were leaked with every encrypted packet received.

rrdtool plugin: Correctly free the cache when shutting down.

network plugin: Be more strict when checking the return value of "setsockopt".

README: Fix a typo.

Merge commit 'tokkee/sh/collectd-4.8' into collectd-4.8

collectd.conf.in: Change the basename of the Perl plugin to "Collectd::Plugins".

collectd-perl(5): Fix the synopsis.

I think this is a left-over from a switch from "Collectd::Plugin" to
"Collectd::Plugins". Thanks to Denis Yagofarov for reporting the bug and
Michael Shigorin for relaying it to me.

configure, src/Makefile: Check for nanosleep() in libposix4 as well.

On, e.g., Solaris 2.6, nanosleep() is only available from that library.

Thanks to Vincent McIntyre for reporting this and providing an initial patch.

README: Fix typo.

README: Fix typo.

snmp plugin: Fix a format string.

openvpn plugin: Fix a copy'n'paste error in the "new naming schema".

collectd.conf(5): Document the "ImprovedNamingSchema" option of the OpenVPN plugin.

netapp plugin: Fix a syntax error in an error message.

Thanks to facade from IRC for the hint.

Merge branch 'collectd-4.8' into collectd-4.9

Conflicts:
src/exec.c

Exec plugin: Update copyright header.

configfile: cf_read_dir: Don't abort reading a directory when reading one file fails.

Processes plugin: Fix a minor typo.

Exec plugin: Add a missing folding marker.

Exec plugin: Set the "noreturn" attribute on the "exec_child" function.

Exec plugin: Reset the "PL_RUNNING" if "fork_child()" fails.

libiptc: avoid strict-aliasing warnings

libiptc: avoid strict-aliasing warnings

In file included from libiptc/libip4tc.c:117:0:
libiptc/libiptc.c: In function ‘__iptcc_p_del_policy’:
libiptc/libiptc.c:826:4: warning: dereferencing type-punned pointer will break
strict-aliasing rules
libiptc/libiptc.c: In function ‘iptc_get_target’:
libiptc/libiptc.c:1650:4: warning: dereferencing type-punned pointer will break
strict-aliasing rules
libiptc/libip4tc.c: In function ‘dump_entry’:
libiptc/libip4tc.c:157:3: warning: dereferencing type-punned pointer will break
strict-aliasing rules
  CC     libiptc/libip6tc.lo
In file included from libiptc/libip6tc.c:112:0:
libiptc/libiptc.c: In function ‘__iptcc_p_del_policy’:
libiptc/libiptc.c:826:4: warning: dereferencing type-punned pointer will break
strict-aliasing rules
libiptc/libiptc.c: In function ‘ip6tc_get_target’:
libiptc/libiptc.c:1650:4: warning: dereferencing type-punned pointer will break
strict-aliasing rules
libiptc/libip6tc.c: In function ‘dump_entry’:
libiptc/libip6tc.c:188:3: warning: dereferencing type-punned pointer will break
strict-aliasing rules

Signed-off-by: Jan Engelhardt <jengelh@medozas.de>
Rebased for collectd/src/owniptc: Alan Pevec <apevec@gmail.com>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

Merge remote branch 'origin/collectd-4.8' into collectd-4.8

configure.in: Check for "htonll".

The first patch adds support to check if the htonl and htonll functions
(which are used in AIX port) are available in configure. Until now the
AIX port used these functions without perform any existence test.

Signed-off-by: Florian Forster <octo@leeloo.lan.home.verplant.org>

memcached plugin: Add hack for AIX compatibility.

The second one port memcached plugin to AIX. Since AIX don't have
MSG_DONTWAIT flag support, we use MSG_NONBLOCK... and we hope to work
fine ever :)

Signed-off-by: Florian Forster <octo@leeloo.lan.home.verplant.org>

oracle plugin: Fix semantics of OCIAttrGet.

Apparently it returns success if it could fetch the parameter value and
doesn't interpret the value itself. So returning NULL when querying for
OCI_ATTR_SERVER_STATUS does not result in an error. Handle this as a
lost connection.

Thanks to Sven for pointing this out :)

.mailmap: Added Rodolphe Quiedeville <rquiedeville@bearstech.com>.

His name was mis-spelled in one commit as Rodolphe Quiedeville*l*.

Merge branch 'collectd-4.8' into collectd-4.9

Conflicts:

src/ping.c

contrib/redhat/collectd.spec: Updated for version 4.9.

I've attached a patch for the RPM specfile to allow it to build collectd 4.9.0,
including Java support.

debian/: Removed ancient Debian package files.

You might want to look at git://git.tokkee.org/pkg-collectd.git instead.

configure.in: Fix three small typos.

Build system: Put Java class files into a .jar file.

Hi,

I was wondering why the java plugin api class files are published under
share/collectd/java. Ideally, a java-plugin.jar file containing all the class
files should get published rather than the individual class files.
Additionally, the jar files IMO should get published to the lib directory where
the so files are sitting rather than the share directory.

Same holds true for jmx class files as well (probably jmx.jar?).

I could work on this if everyone is okay with it.

Regards
Amit

src/utils_cmd_listval.c: Fix a memory leak.

Thanks to Peter Warasin for figuring this out. His "garbage collector" patch
may replace this solution in future versions of collectd.

Bugfix: correctly handle spurious wakeups in pthread_cond_timedwait

This fixes problems storing data in rrd files on NetBSD 5.0 with more
than one CPU.

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

src/utils_format_json.c: Use "isfinite" rather than "isinf".

Makes it a bit easier to read.

src/collectd.h: Add fallback implementations of "isfinite" and "isinf".

utils_format_json: serialize nan and inf as null, as per JSON spec

I'm deserializing JSON output from the write_http plugin using ruby-yajl.
yajl was puking on the literal value nan encoded in the output.

[
    {
        "plugin": "memcached",
        "interval": 10,
        "host": "myhost",
        "values": [
            nan,
            5
        ],
        "time": 1265239180,
        "plugin_instance": "",
        "type_instance": "",
        "type": "ps_count"
    }
]

After some research, ECMA-262 15.12.3 says nan and infinite numbers
aren't representable in JSON and should be serialized as the string
null. I figure any strictly-compliant JSON parser will fail on parsing
JSON data containing nans as emitted by collectd's utils_format_json
routines.

This patch makes collectd's JSON output compliant in the case of
infinite or nan gauge values.

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

ping plugin: Improved some error messages.

contextswitch plugin: Handle large counter/derive values correctly.

Using "strtoul" for derive_t values fails for large counter values.
Thanks to Martin Merkel for reporting this :)

collectd{-python,.conf}(5): Fixed some typos.

Thanks to lintian(1) for catching those!

collectd{-perl,.conf}(5), {my,postgre}sql plugins: Fixed some typos.

Thanks to lintian(1) for catching (some of) those! :-)

apache plugin: Fix collection of the "apache_connections" value with Apache 2.*.

Thanks to Gary (bug tracker) for pointing this out :)

contrib/collection.cgi: Added ability to hide specified types.

It is useful when you don't want to see many graphs. How to use the patch:
apply it to collection.cgi and add lines with keyword ,,dontshowtype'' to
/etc/collectd/collection.conf:

dontshowtype: "if_errors"
dontshowtype: "if_packets"

Then you will see only if_octets types in "interface" plugin page.

Via Debian bug #566199.

src/filter_chain.c: Make info-message more verbose.

Use Curl AnyAuth (digest+basic) instead of just digest

Merge commit 'tokkee/sh/collectd-4.8' into collectd-4.8

exec plugin: Improved some error messages.

contrib/exec-munin.px: Add GPLv2 header.

Merge branch 'collectd-4.8' into collectd-4.9

Conflicts:
ChangeLog

ChangeLog: Added fix in the processes plugin.

processes plugin: Fix handling of the ps_disk_{octets,ops} types.

There values were stored in a `long', then `counter_t', then interpreted as
`derive_t'. Changed that to `derive_t' only. Hopefully this fixes a bug
reported by Mark Moseley.

ChangeLog: Updated for version 4.9.1.

ChangeLog: Updated for version 4.8.3.

Merge commit 'tokkee/sh/collectd-4.8' into collectd-4.8

collectd.conf(5): Rephrase a weird sentence in the cURL-JSON documentation.

collectd.conf(5): Remove trailing whitespace.

types.db(5): Mention the types "ABSOLUTE" and "DERIVE" as well.

Merge branch 'collectd-4.8' into collectd-4.9

Merge branch 'collectd-4.7' into collectd-4.8

bindings/perl/lib/Collectd.pm: Add forgotten "config" callback type.

Thanks to "ntenev" for reporting the issue.

Merge commit 'tokkee/sh/collectd-4.9' into collectd-4.9

Merge branch 'collectd-4.8' into collectd-4.9

rrdtool plugin: Correctly initialize the ->random_variation member.

This lead to huge negative values in that member, causing values to be only
written to upon exit. D'oh!

apache plugin: Fix a segmentation fault in the config handling of VerifyPeer / VerifyHost.

Hello,

Today I upgraded a machine from 4.5.2 to 4.9.0. Everything went fine, but
the new version segfaults immediately at startup. With some help from gdb
I tracked down the problem to the apache module. It will occur if one
tries to use the VerifyPeer/VerifyHost options (I guess few people use
these as I don't see complaints?)

When the plugin attempts to read the value of a boolean option, instead of
using the pre-parsed int value, it misdetects the type of the option
(seems a mechanical mistake) and tries to use the string pointer, which is
actually NULL.
I'm sending a patch, it's actually shorter than the description ;)

Thanks to all developers and maintainers of collectd for working on it.
Have a nice holidays.

Monitorus.pm: Put the plugin into the "Collectd::Plugins" namespace.

… rather than "Collectd::Plugin".

bindings/Makefile: Added Monitorus.pm to EXTRA_DIST.

src/collectd.conf: Added missing plugins and config options.

collectd.conf: Added "LoadPlugin Monitorus/OpenVZ" in the "perl" config.

README: Added missing plugins.

src/collectd.conf.in: Added an example config for the python plugin.

src/collectd.conf.in: Added an example config for the netapp plugin.

src/collectd.conf.in: Add example config for the routeros plugin.

routeros plugin: Use the node name for the "host" field.

Using hostname_g doesn't make much sense when you can define multiple routers,
so this is a bug because it doesn't allow specifying multiple routers as documented.

contrib/snmp-probe-host.px: Implemented the “--exclude” option.

collectd-snmp(5): Comma fixes.

Annoying signal documentation.