tokkee.org

Fix compilation error with plugin_register_shutdown

Compiling collectd-4.10.2 on an OpenSuSE 11.3 system causes the
following error:
  snmp.c: In function ‘module_register’:
  snmp.c:1620:3: error: passing argument 1 of ‘plugin_register_shutdown’ discards qualifiers from pointer target type
  plugin.h:275:5: note: expected ‘char *’ but argument is of type ‘const char *’

Convert plugin_register_shutdown's name argument from 'char *' to
'const char *' in order to match the other plugin_register_*
functions.

Signed-off-by: Bruno Prémont <bonbons@linux-vserver.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

dispatch proper values in Python write plugin

Fixes the Python write callback so the appropriate value is dispatched to
Python. Previously, the code only looked at the first element of a data set
to determine which value type (GAUGE, COUNTER, etc) to dispatch. If your data
set consisted of multiple values of different types, then the Python write
plugin was receiving bad values for the elements at position n > 0 whose type
was not the same as that at position 0.

processes plugin: Call kvm_close() after kvm_geterr().

collectd.conf(5): Mention MySQL privileges required for repl stats.

collectd.conf(5): Be more verbose about how to access a notification daemon.

filter_chain: Use a complaint to report plugin_write()'s ENOENT error.

In most cases this is a permanent error, so using the complain mechanism
avoids spamming the logs.

src/plugin.c: Print a more verbose error message if lt_dlopen() fails.

Since the Debian package doesn't depend on the libraries used by the
collectd plugins, some plugins may fail to load. ltdl reports this with
the very confusing error message "file not found". Since the plugin is
in fact available, many users don't realize a dependency is missing and
assume collectd is looking in the wrong directory -- and they are hardly
to blame for this.

This commit introduces a lengthy error message which hopefully points
users into the right direction.

exec plugin: Remove useless preprocessor stuff.

ntpd plugin: Report failures of "ntpd_do_query" as *errors*, not debug messages.

src/utils_threshold.c: Fix creation of percentage notifications.

Joey Hess has reported a problem when creating notifications from
percentage thresholds. Because the (percentage) minimum value is
compared to the (raw) DS value, the following message is possible:

  Message: Host XXX, plugin df type df (instance root): Data source
  "free" is currently 1773072384.000000. That is above the warning
  threshold of nan%.

A new section will handle this case correctly. In the inverted case, the
problem should not exist.

dns plugin: Include <pcap-bpf.h> if available.

java plugin: Improve an error message.

src/common.c: parse_value: Fix assignment of derive and absolute values.

configure.in: Fix a typo in an AC_ARG_WITH description.

Bump version to 4.9.4; Update ChangeLog.

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

netapp: Fixed incompatible changes between NetApp Release 7.2 and 7.3 that prevented collecting SIS data

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

Fixed RedHat init script to shut down collectd on reboot/halt.

collectd.conf(5): Document the dangers of messing with the interval setting.

processes plugin: Don't complain about a failed open(/proc/$PID/cmdline)

... if the error is ENOENT. This merely means we were too slow and that
we can safely ignore the process.

src/utils_rrdcreate.c: Remove a too strict assertion.

We don't actually check anywhere that vl->time is greater than 10, so we
cannot assure this.

README: Add libperfstat as an optional dependency.

netapp plugin: Fixed a bug that prevented the configured interval to be passed on to the dispatch function.

Add sys.argv. Not too many programs consider the possibility that it might not exist resulting in unhandled exceptions.

Fixed possible memory leak in case of broken match_value configs.

curl_json plugin: Restore struct when handling error.

src/collectd.h: Do not include <sensors/sensors.h> here.

Fix errno thread-safety under AIX

Unlike Linux or Solaris, errno under AIX is not thread-safe by
default.

This patch sets _THREAD_SAFE_ERRNO when AIX is detected in order to
force the thread-safe implementation of errno. Without this, calls
like stat() in the rrdtool plugin fail with errno incorrectly set,
leading to the inability to create previously absent rrd files.

Maybe _THREAD_SAFE should be set instead, to prevent other possible
threads-related problems, but this is enough to scratch my current
itch...

Here is the relevant part of /usr/include/errno.h on AIX:

 #if defined(_THREAD_SAFE) || defined(_THREAD_SAFE_ERRNO)
 /*
  * Per thread errno is provided by the threads provider. Both the extern
  * int
  * and the per thread value must be maintained by the threads library.
  */
 extern  int     *_Errno( void );
 #define errno   (*_Errno())

 #else

 extern int errno;

 #endif  /* _THREAD_SAFE || _THREAD_SAFE_ERRNO */

Signed-off-by: Aurelien Reynaud <collectd@wattapower.net>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

collection3: Add graph for "ps_disk_octets".

notify_email plugin: Check for "session == NULL" and return gracefully.

I don't know how libesmtp handles NULL pointers and I don't want to take
my chances.

notify_email plugin: Serialize all accesses to libesmtp using a mutex.

collection3: fix multiple hosts selection issue

When selecting multiple hosts in collection3's hosts list, and with some
plugins only (the ones that use a specific resource name such as memory
or tcpconns), the resulting graph list will be limited to one single
graph (instead of one per host).

This patch addresses this issue by modifying the name of the hash key
in the group_files_by_plugin_instance function, making it less prone
to name collisions by prefixing it by the host name.

Comments and enhancements will be welcome.

Signed-off-by: Jerome Oufella <jerome.oufella@savoirfairelinux.com>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

network plugin: Use the complain mechanism to inform the user about ignored signatures.

src/utils_tail: Remove newline characters at the end of a line.

The tool is supposed to work line-based, so a) the newline character is
implicit and b) imho, does not belong to the string which is then further
processed.

Also, this allows '$' to match the end of a line when applying a regex to the
line, no matter if the REG_NEWLINE flag has been used when compiling the regex
or not.

utils_match: Let '^' and '$' affect lines rather than the full string.

The regcomp() flag REG_NEWLINE is required for that, even though regex(7)
might make people think otherwise.

src/collectd.conf.in: Move "TimeToLive" before the "Server" and "Listen" statements.

Thanks to Renaud Chaput for pointing this out!

src/utils_heap.c: Fix calculation of the parent's index.

This resulted in the "upwards" reheap function to return prematurely,
leaving the heap condition violated.

src/plugin.c: Improve an info message.

Based on a patch by Sebastian, which didn't use the handy "FORMAT_VL"
macro.

collectd.conf(5): Document he "Derive*" and "AbsoluteSet" DSTypes of the tail plugin.

src/utils_match.c: Use strtoull(3) for counter and absolute data sources.

src/utils_match.h: Fix a bug preventing derive values from being handled correctly.

Due to the bit-wise check, the value for derive (0x30) matches the check
for gauge (0x10), too. This commit fixes the behavior by assigning other
numeric values to the defines.

src/configfile.c: Continue parsing config files if stat'ing one file fails.

curl json: Fix checking the response code.

curl_json plugin: Use a more descriptive define as return value.

collectd-perl(5): Mention that "Globals" should be enabled.

collectd.conf(5): Documents LoadPlugin's "Globals" option.

curl_json plugin: Use the "number" callback of libyajl.

The "integer" callback only works with "long"s, which are 32bit on
x86 and other 32bit architectures. The "number" callback gets the raw
string for us to parse ourselves – honoring the data source type in the
process.

The "integer" and "double" callbacks have been removed, since they are not
used if the "number" callback is present.

ChangeLog: Fix the release date for 4.9.3.

Bumped version to 4.9.3; Updated ChangeLog.

src/plugin.c: Check "read_loop" when returning from "pthread_cond_timedwait".

Otherwise it may take up to $Interval seconds until all read threads
shut down. This bug was introduced in version 4.8.4 (commit dbe1a7d).

Remove usage of obsoleted XSI extensions wrt. the "test" command.

Fix usage of Solaris /usr/bin/echo

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

df plugin: Remove the complaint code again.

Calls to the plugin are delayed when an error is returned anyway, so it's
better to have an error message in addition to the "has failed, will suspend"
message generated in src/plugin.c.

df plugin: Report an error if "cu_mount_getlist" fails.

collection3: df_complex graph: Add "reserved" space.

collection3: Add Windows data sources to the "memory" graph.

collection3: Add graphs for "bitrate", "signal_power", "signal_quality", and "snr".

bindings/java/Makefile.am: Fix rebuilding of Java plugins.

bind plugin: Fix building with --enable-standards.

Merge remote branch 'tokkee/sh/collectd-4.9' into collectd-4.9

contrib/GenericJMX.conf: Use the "jmx_memory" type.

collectd2html.pl: Further improvements to the XHTML support.

Signed-off-by: Sebastian Harl <sh@tokkee.org>

collectd2html.pl: Produce XHTML output.

… as requested in Debian bug #575029.

Signed-off-by: Sebastian Harl <tokkee@debian.org>

collectd2html: Fixed recursive mode.

Also, improved default settings to match the collectd4 series.

Signed-off-by: Sebastian Harl <tokkee@debian.org>

powerdns plugin: Fixed communication when talking to pdns 2.9.22.

powerdns version 2.9.22 (and above) require a newline at the end of
commands sent to the server.

Signed-off-by: Sebastian Harl <tokkee@debian.org>

snmp.c: compilation fixes for AIX

The C and C++ standards allows the character type char to be signed or
unsigned, depending on the platform and compiler. Most systems,
including x86 GNU/Linux and Microsoft Windows, use signed char, but
those based on PowerPC and ARM processors typically use unsigned char.

This patch fixes a "comparison is always true" warning on AIX (powerpc)
which leads the compilation to abort because of the -Werror flag.

Being unsigned by default, a char is always >0.

Signed-off-by: Aurelien Reynaud <collectd@wattapower.net>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

processes.c: fix uninitialized variables warnings

Signed-off-by: Aurelien Reynaud <collectd@wattapower.net>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

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.