Merge branch 'collectd-4.10' into collectd-5.0

Conflicts:
src/df.c

Change-Id: Ie7c9fd4a20cc356bb53b135bf2ca2d5162a8a953

contrib/exec-nagios.px: Implement the "NRPEConfig" option.

bind plugin: Fix use of the "QType" types.

This should resolve GitHub issue #21.

Change-Id: I67e4b440a6fbc8a8608fa672fb3320b05f9c0a12

df plugin: Fix a compiler issue with Mac OS X 10.7.

Xcode Version 4.1 (11C74), GitHub issue #24

Change-Id: Ic6f49a95868c7f77bfb19f9c8afc87a97a1bf670

AMQP plugin: Support sub-second sleep intervals.

Change-Id: I9afcd8d17f1251b0765dfbdae5ed8581b73b15fe

AMQP plugin: Fix sleeping of "interval_g seconds".

Since "interval_g" was changed to "cdtime_t", the plugin would sleep 2^30 times
as long as intended. D'oh! Thanks to Xavier for reporting this!

Change-Id: I8e9992307a1fdbb2df799a5359eb6695409684df

Bump version to 5.0.3; Update ChangeLog.

Change-Id: I3c73ac8b3ac73227e20ea32d052f5bea1575e29a

Merge branch 'collectd-4.10' into collectd-5.0

Conflicts:
ChangeLog
version-gen.sh

Change-Id: Id72b28eb1432835ab710d1f551bb886d0638e079

Update ChangeLog.

Change-Id: I41518e6e48c089e79919d1bc65b03744872bcec0

exec plugin: Replace tabs with spaces to fix indentation.

Change-Id: I6e623f237c1d79007bd471732476a4e32dafde97

network plugin: Fixed incorrect collectd warning when configuring network plugin.

Cosmetic only

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

exec plugin: incorrectly exited select loop on signal.

Sometimes dead modules won't be started up correctly or running ones
would not generate stats anymore.  Problem manifests more often under
load and on multicore hw

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

configure.in: Use the `--libs_r' option of mysql-config.

Rather than the `--libs' option.

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

Bump version to 4.10.6; Update ChangeLog.

Change-Id: I95bfb6e742ba7ad9e2b9c89dc990cd4bf573dbfa

Build system: Remove superfluous C(PP)FLAG.

Change-Id: Ic2f665869f863b1e0dca9b5ccde2803715386a3c

Build system: Use pkg-config to configure libiptc.

This hopefully fixes Goobuntu Bug #375733.

Change-Id: I0a4215351ce836fb62418cbd40bee27fd695d9da

Merge branch 'collectd-4.10' into collectd-5.0

version-gen.sh: Switch to bash due to bashisms.

Change-Id: I8480afad2d157382fb7e5ff35785e97066c8553c

Revert "interface plugin: Fix for bogus interfaces under Solaris."

This results in problems on aggregated interfaces, amongst other things.
I guess Solaris users will need to use the "IgnoreSelected" option to
get rid of "bogus" interfaces.

This reverts commit e7a75874a24c1bf19931d5b4b423a0daa7f9be6b.

network plugin: Document the standard buffer size with more detail.

Change-Id: I48e61fc2613ca50c477794167f180e3b4f4fc673

Merge branch 'collectd-4.10' into collectd-5.0

Merge remote-tracking branch 'tokkee/sh/collectd-5.0' into collectd-5.0

Minor doc fixes for the Python plugin.

Change-Id: I233d0ae4ec035c74811ef33aa119573066c9a3f6

Merge pull request #2 from trenkel/collectd-4.10

Even more fun than memory leaks: Memory corruption.

python: Fixed crash bug in case the plugin had more than one configuration block.

oracle plugin: Fix use of the OCIErrorGet() function.

Change-Id: I9899b98517fe0c239bffcf7a75681560029aa2ba

common: check_create_dir(): Support symlinks as well.

Previously, the following situation would cause an endless look (as reported
by Michael Prokop in Debian bug #619123): the (CSV or RRD) datadir is a
symlink pointing to a non-existent target.

With this patch applied, check_create_dir() fails with "<file> exists but is
not a directory".

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

build system, ipvs plugin: Check for linux/ip_vs.h as well.

Also, try to find ip_vs.h without using KERNEL_CFLAGS at first. The Linux
kernel headers should not be used directly -- in current kernel versions this
generates warnings leading to build failures because of -Werror.

Merge branch 'collectd-4.10' into collectd-5.0

Change-Id: I8fea9338ac6490b2a76530a58c6bd36ec17d021a

Update ChangeLog.

Change-Id: I7361ea08e1695b2928463150df36d385aa51299c

oracle plugin: Fix a memory leak.

Parameters must be freed using OCIDescriptorFree. D'oh!

Thanks to Rena Vetterling for helping me hunt down this problem!

Change-Id: Ic374ae13085c8d2afbdf9a3c0ceef92b485e7723

oracle plugin: Improve the error printing function.

Change-Id: Ie52c80bd4a686b403037c97383176adf3e701419

Bump version to 5.0.2; Update ChangeLog.

Change-Id: I8e564e51d9e9f3dc847811fd0d51fbf0079c20ba

Merge branch 'collectd-4.10' into collectd-5.0

Conflicts:
ChangeLog
src/df.c
src/plugin.h
version-gen.sh

Change-Id: Id7a31088cdb3371534d5c1e6a690464f3343543d

Bump version to 4.10.5; Update ChangeLog.

Change-Id: I900278064970e0c2d2bf471f4f80c7672aa8211f

df: Set negative free space values to zero.

Some file systems (eg. UFS) may report negative free spave values.
Previously this caused underflow, but now it is prevented by setting
negative values to zero.

Change-Id: I821adb8a6d7a40dcbd6c65ab3541fbcd714f8aba

Conflicts:

src/df.c

network plugin: Use the new plugin_dispatch_values_secure().

Change-Id: I2cc65a36366c6c714cd79c2905879bce313efe31

src/plugin.[ch]: Add plugin_dispatch_values_secure().

Change-Id: I1f7403b2e82edd099f0168d0a0735cd18f8ce05a

src/meta_data.[ch]: Add meta_data_clone().

Change-Id: I8449e27c908cbe75ea4ea3b8dd4df556f9aec9e7

Merge "Fix case when availrmem < freemem" into collectd-4.10

Fix case when availrmem < freemem

Change-Id: I03b188afeb2357e7b8c426bd0c6daff06fda2c4f

Conflicts:

src/memory.c

Merge pull request #13 from insom/improved-migration

Migration: Ensure destination directories exist when moving RRDs

python: Fixed memory leaks during dispatch.

Migration: Ensure destination directories exist when moving RRDs

If you have not run collectd 5 before, then when type instances
are migrated to plugin instances the destination directories
may not already exist. For example:

mv "./test/df/df-mnt-dlna.rrd" "./test/df-mnt-dlna/df.rrd"

This one-liner does a mkdir -p of the destination before
attempting to move the file.

src/common.c: parse_value: Be more verbose about parsing problems.

Change-Id: I784ec4ee9a1e89a82fdc9da54ed1fafd3d9c3e57

collectd-perl(5): Fix example Perl script.

Thanks to Sergey for catching this!

Change-Id: I4bbb609079734cd54d68f77681ea742b4eb83239

This improves solaris memory plugin granularity.

Locked memory is now only tracking non-kernel locked memory.
We add kernel and unusable memory.
Source: http://www.brendangregg.com/K9Toolkit/swapinfo

Change-Id: I4f4d281e991068dc845d8c3615d285fc2a3296c8

use 64bit counters if available

Merge "Fix file:// not working as return code is 0 and not 200, as fixed beforehand in curl_json. Remove mention of http in documention as file also works now (and ftp most likely too)" into collectd-4.10

Fix file:// not working as return code is 0 and not 200, as fixed
beforehand in curl_json.
Remove mention of http in documention as file also works now (and ftp most likely too)

Change-Id: Iccb35e6b8c831f555dc5bbd17fa8d04672e13eb3

Merge "interface plugin: Fix for bogus interfaces under Solaris." into collectd-4.10

ChangeLog: Remove blank line.

Change-Id: Ief9cabb148d75b79e0764d629c571ee9dc3d7d3b

Merge branch 'collectd-4.10' into collectd-5.0

Conflicts:
ChangeLog

Change-Id: I65f02dbcad46460f18f356bc8d32eb15a7a352c2

ChangeLog: Add thanks to Peter Green.

Change-Id: Ie392f7dfc7d637c92e286b0bf6d5d32877f70f76

Bump version to 5.0.1; Update ChangeLog.

Change-Id: I01466d5fb5c0d5c08a48d7c7e7a81239960a2d3c

Merge branch 'collectd-4.10' into collectd-5.0

Conflicts:
ChangeLog
src/collectd.conf.in
src/curl_json.c
version-gen.sh

Change-Id: I313c0d3c73d47481baa0a3b38bcdf508f1d4f8d8

Bump version to 4.10.4; Update ChangeLog.

Change-Id: Ie23cf45f929c52e94e968c7129d24aca7a287dd5

interface plugin: Fix for bogus interfaces under Solaris.

Under newer versions of Solaris, the method for finding network interfaces
results in 'collectd' gathering statistics on an interface called 'mac', along
with 16 interfaces named 'wrsmd0' through 'wrsmd15' (which are for WCI Remote
Shared Memory). The 'mac' interface is especially problematic because it
occurs multiple times, which results in warnings from rrdtool about insertions
with the same timestamp.

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

collectd.conf.in: Use the "Globals" option for perl and python.

Change-Id: Icaccaab601b91bc829c4da94c5344e0d766735bd

Add RPM .spec file and startup script for AIX

Change-Id: Ic6187891e2014d84f8b2926df1c8b2012f26923f
Signed-off-by: Aurelien Reynaud <collectd@wattapower.net>
Signed-off-by: Florian Forster <octo@collectd.org>

Merge "curl_json plugin: Be more clever building the type instance." into collectd-4.10

Merge branch 'collectd-4.10' into collectd-5.0

Conflicts:
src/irq.c

Change-Id: Ie6c78234f07cc5b7bac7e6914813e9545ce1b1af

irq plugin: Implement collection of non-numeric interrupts.

Hi all!

I've created a patch that enables collectd to collect all irqs present in
/proc/interrupts, with numeric and string names (NMI, LOC, RES, etc).

Two notices:
a) Unfortunately I was unable to build collectd from git repo (build.sh
   failures), therefore I created this patch for latest stable release: 4.10.1
b) I am fairly unexperienced in C programming, so please inform me gently of my
   mistakes:) But I am already using a patched version and it works fine.

Any feedback on code quality or recommendations are welcome.

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

processes plugin: Fixed handling of ProcessMatch regexes containing spaces.

Previously, something like 'ProcessMatch name "My Regex"' would have been
interpreted as three values. This was caused by using the old, non-complex
config interface which joins all config values using a space and passing the
resulting string to the plugin. The processes plugin used to use strsplit() to
re-gain a list of all values, obviously ignoring any quoting that might exist.

This has been fixed by using the complex config interface, which passes all
values as an array of strings and thus honoring the quoting handled by
liboconfig.

Merge "exec plugin: Fixed incorrect typecast of 64-bit timestamp to unsigned int (collectd-5.0)" into collectd-5.0

exec plugin: Fixed incorrect typecast of 64-bit timestamp to unsigned int (collectd-5.0)

Change-Id: Ia99285c8158828760e74083ed6ea54112fa8a156
Signed-off-by: Alexander Kovalenko <alexanderk23@gmail.com>
Signed-off-by: Florian Forster <octo@collectd.org>

curl_json plugin: Be more clever building the type instance.

When the key of the parent object is empty, don't use it. The previous code led
to type instances starting with a dash, e.g. "-foo".

Also, be more verbose when unable to resolve types.

Change-Id: Ib5f55efe1acc1e819ac3189b6780a4e998bf9c9f

Varnish plugin: Fix gauge vs. derive issues.

Some of the values are submitted using the wrong submit function.

Change-Id: Icf82844e54605d46cce8e41d2678aa4bca1cb2c1

Merge "fix xff excessive rounding" into collectd-4.10

.gitignore: Added collectdctl.

Correct a missing yajl1/2 check

Change-Id: Ie3e0380721d88e54cb5630e1c3ecb613790b32b3

src/plugin.c: add info to 'Invalid value list' error message

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

collectd_unixsock.py: Fix infinite wait.

raise a KeyError if getval() or getthreshold() unixsock returns replies an
error because of request of an unknown identifier

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

rrdtool plugin: Fix warning about unused variable "stepsize".

Change-Id: Ibd4dd3f17db62b10b96c59f37b66941ddc5ea7b0

Merge branch 'collectd-4.10' into collectd-5.0

Conflicts:
src/curl_json.c
src/libvirt.c

Change-Id: I0852495b416435fa3cfd36068d967e0cd5ff689f

configure: Link with ip4tc and ip6tc, too.

This solves this problem:

octo@leeloo:/tmp $ gcc -o conftest -g -O2     conftest.c -liptc -ldl
/usr/bin/ld: /tmp/ccuJEDqP.o: undefined reference to symbol 'iptc_init'
/usr/bin/ld: note: 'iptc_init' is defined in DSO /lib64/libip4tc.so.0 so try adding it to the linker command line
/lib64/libip4tc.so.0: could not read symbols: Invalid operation
collect2: ld returned 1 exit status

Why the iptables plugin compiles without this is beyond me.

Change-Id: If615bee0c9524d907ca1834d5c337ed50c625c5b

fix xff excessive rounding

Hi,

When generating arguments for rrd_create, the previous
format string "%.1f", was rounding up the value too aggressively.

ex: for xff==0.9999 the result would be -> 1.0 (invalid value for xff!)

The new format string is "%.10f", which lead up to 10 digits after the
dot, enough to be close to the double type limit, and so handling
correctly the previous example.

Change-Id: I7c6cc55b0d43beb5ef351bb04b67f4628249c737

configure.in: Add AC_CONFIG_AUX_DIR for libltdl.

mysql plugin: Remove sideeffect-free function call.

notify_email plugin: Remove useless function and fix indentation.

Many build fixes that turned up with GCC 4.6.

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

src/csv.c: use a bigger buffer

The CSV plugin formerly used a relatively small output buffer. If you
had a large dataset, you would always get error -1 because the line
length was be too long. This patch extends the buffer.

Signed-off-by: Colin McCabe <colin.mccabe@dreamhost.com>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

src/utils_format_json.c: Correctly format the time of value lists.

Users should get epoch, not our internal format.

curl_json plugin: Add compatibility code for libyajl v2.

Thanks to "spupykin" of the Arch Linux project on whose patch this commit is
loosely based.

collectd.conf: Added a (commented out) 'LoadPlugin target_v5upgrade' line.

README: Added 'write_redis' plugin.

Various plugins: Set the cURL option "CURLOPT_NOSIGNAL".

This fixes problems with version 7.21.2 (and later) of libcurl that are due to
collectd being a multi-threaded program.

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

src/meta_data.c: Free a leaking mutex.

This problem primarily manifested on FreeBSD but may well effect other
systems as well.

Thanks to Tomas Krasnican for reporting this problem.

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

df plugin: Improve error message.

contrib/migrate-4-5.px: "df" does not have COUNTER DSTs.

contrib/migrate-4-5.px: Remove useless "OutDir" option.

notify_desktop plugin: Add compatibility code for libnotify 0.7.

The current code in src/notify_desktop.c is not compatible with the new
libnotify 0.7 API

This:

  notification = notify_notification_new (summary, n->message, NULL, NULL);

Should now be:

  notification = notify_notification_new (summary, n->message, NULL;

As in, one argument less for notify_notification_new.

But we can't just remove it or it'll break compability with libnotify
0.4.x and 0.5.x.

This piece of code sets dummy NOTIFY_CHECK_VERSION for libnotify-0.4.x,
because NOTIFY_CHECK_VERSION was added only in 0.5.x:

  #ifndef NOTIFY_CHECK_VERSION
  #define NOTIFY_CHECK_VERSION(x,y,z) 0
  #endif

Then we can freely use NOTIFY_CHECK_VERSION, and it will work with all
of 0.4, 0.5 and 0.7 series:

  #if NOTIFY_CHECK_VERSION (0, 7, 0)
  do something cool
  #endif

I'm attaching the working patch.  It's been tested with all of the
mentioned versions, as well as it's in active use at Gentoo Linux.

Thanks, Samuli

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

src/collectd.h: Define "_Bool" to "int" if the C compiler doesn't know the type.

ChangeLog: Add release date.

Merge branch 'collectd-4.10' into collectd-5.0

Conflicts:
ChangeLog
src/libvirt.c
src/plugin.c
src/plugin.h
version-gen.sh

Bump version to 4.10.3; Update ChangeLog.

Merge branch 'collectd-4.9' into collectd-4.10

Conflicts:
ChangeLog
version-gen.sh

Bump version to 4.9.5; Update ChangeLog.

libvirt plugin: Correctly check the status code of virDomainGetVcpus().

The status code is less than zero on failure and the number of vCPUs
otherwise. Thanks to "JLPC" for pointing this problem out.

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>