Merge pull request #947 from mfournier/avoid_using_curlopt_userpwd

avoid using CURLOPT_USERPWD when possible

amqp: add documentation bits for ConnectionRetryDelay feature

amqp: disable reconnection delaying feature by default

amqp: coding-style consistency

AMQP connection failure leads collectd to use 100%CPU

Summary:
Issue : https://ipm.bloomberg.com/jira/browse/SS-103
Solution : When amqp connection fails, it retries for every message, added a fix to retry only after a delay. A new configuration "ConnectionRetryDelay"  has been introduced with default value as 60 seconds.

Test Plan:
1. Tested without the new configuration
2. Tested with the new configuration

In both the cases, verified connection is retried only after the delay and made sure collectd is not taking 100% cpu.

Reviewers: skhajamo

Reviewed By: skhajamo

CC: arcyd
Differential Revision: https://all.phab.dev.bloomberg.com/D149956

Conflicts:
src/amqp.c

Merge pull request #965 from rubenk/powerdns-update-auth-stats

powerdns: update plugin with stats for pdns 3.4.3

Merge pull request #959 from rubenk/fix-ceph-plugin

Ceph: do not define _BSD_SOURCE

write_tsdb plugin: Fix name generation even more.

There were more cases in which "type" or "type_instance" were omitted.

Fixes: #945

write_tsdb plugin: Add type and type_instance to the metric name in any case.

Previously, the generated names were wrong, especially when no
plugin instance was used for a metric.

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

powerdns: update plugin with stats for pdns 3.4.3

Ceph: define _DEFAULT_SOURCE next to _BSD_SOURCE

Fixes:
make[3]: Entering directory '/home/ruben/src/collectd/src'
  CC       ceph_la-ceph.lo
In file included from /usr/include/stdio.h:27:0,
                 from ./daemon/collectd.h:34,
                 from ceph.c:26:
/usr/include/features.h:148:3: error: #warning "_BSD_SOURCE and
_SVID_SOURCE are deprecated, use _DEFAULT_SOURCE" [-Werror=cpp]
 # warning "_BSD_SOURCE and _SVID_SOURCE are deprecated, use
 # _DEFAULT_SOURCE"
   ^
cc1: all warnings being treated as errors
Makefile:4120: recipe for target 'ceph_la-ceph.lo' failed
make[3]: *** [ceph_la-ceph.lo] Error 1
make[3]: Leaving directory '/home/ruben/src/collectd/src'
Makefile:5096: recipe for target 'all-recursive' failed
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory '/home/ruben/src/collectd/src'
Makefile:3142: recipe for target 'all' failed
make[1]: *** [all] Error 2
make[1]: Leaving directory '/home/ruben/src/collectd/src'
Makefile:551: recipe for target 'all-recursive' failed
make: *** [all-recursive] Error 1

Merge pull request #838 from yogeswaran/histogram

statsd histogram to support more than 1 second

Merge pull request #953 from rubenk/powerdns-do-not-look-past-the-end-of-array

Powerdns plugin: fix logging on failed lookup

Merge pull request #954 from rubenk/remove-outdated-advice

Setting Globals to true is not needed anymore

clean.sh: take new 'src/daemon' dir in account

snmp plugin: add hostname to "csnmp_value_list_to_value" error message

Fixes #952

RPM specfile: add ceph plugin

README/AUTHORS: mention ceph plugin

Merge remote-tracking branch 'origin/pr/598'

Conflicts:
src/collectd.conf.in
src/types.db

Remove superfluous debug statements

Setting Globals to true is not needed anymore

003c6e86b480 made this happen by default

Powerdns plugin: fix logging on failed lookup

Don't look past the end of the array but check the boundary first.

snmp plugin: add hostname to "csnmp_instance_list_add" error message

Fixes #939

avoid using CURLOPT_USERPWD when possible

CURLOPT_USERPWD chokes on colons inside usernames or passwords, so use
CURLOPT_USERNAME and CURLOPT_PASSWORD if curl 7.19.1 or newer is found.

Follow-up to 0af75dc13 for the rest of the plugins based on libcurl.

Merge branch 'collectd-5.4'

Conflicts:
src/Makefile.am
src/write_http.c

Merge branch 'collectd-5.3' into collectd-5.4

Merge branch 'collectd-4.10' into collectd-5.3

Conflicts:
src/collectd.conf.pod

Merge pull request #890 from trenkel/master

Fix compiler warnings with Python3.

perl plugin: plugins are linked with *_LIBADD and LIBS not *_LIBS and LIBS

Conflicts:
src/Makefile.am

libperl's ldopts include libaries and therefore should go in LIBS not LDFLAGS

Without this configure fails compile its libperl test program because
-lperl comes before the test program on the compiler command line.

Conflicts:
src/Makefile.am

Merge pull request #390 from radford/libperl-ldopts

libperl's ldopts include libaries and therefore should go in LIBS not LDFLAGS

modbus: rename register types once more

As I was looking over my config files again, ModbusRegisterType
kind of stuck out like a sore thumb.

Modbus does talk about "read holding" and "read input" as "commands";
perhaps renaming it as this patch does is more intuitive, so that all
of the Modbus register configurations start with Register.

I'm not wedded to it, but if you like it & agree, here you go.

Thanks,
-Eric

Merge pull request #908 from vincentbernat/fix/libstatgrab4

libstatgrab: fix detection of libstatgrab >= 0.90

Merge pull request #902 from mfournier/write_http-node-blocks

write_http: deprecate <URL> blocks in favor of <Node>

Merge pull request #899 from mfournier/write_http-multiple-destionations

write_http: make callback names context-dependent

RPM specfile: doc clarification

ipmi: use SIGIO instead of SIGUSR2 to avoid conflict with JVM

Java uses SIGUSR2 to suspend/resume threads. The OpenIPMI plugins also
need a signal to resume its event loop when setting a timer. They can't
both use the same signal. We ask OpenIPMI to use SIGIO instead.

This should fix #114.

write_kafka plugin: Topic property not being set

Merge pull request #910 from vincentbernat/fix/ipmi-dont-use-sigusr2

ipmi: use SIGIO instead of SIGUSR2 to avoid conflict with JVM

ipmi: use SIGIO instead of SIGUSR2 to avoid conflict with JVM

Java uses SIGUSR2 to suspend/resume threads. The OpenIPMI plugins also
need a signal to resume its event loop when setting a timer. They can't
both use the same signal. We ask OpenIPMI to use SIGIO instead.

This should fix #114.

Merge pull request #906 from obazoud/patch-1

Wording automake

added doc CollectStatistics to section rrdcached

Fixes #907

Conflicts:
src/collectd.conf.pod

mysql: document defaults to boolean config options

libstatgrab: fix detection of libstatgrab >= 0.90

Ensure linking is done after compilation of the test program, otherwise
symbols cannot be found. `LDFLAGS` and `CFLAGS` are placed before the
test program on gcc command line while `LIBS` is placed after. Even if
in our case, `LDFLAGS` also contain the lib, being put before the
program while the linking phase is done with the same command leads to
`sg_init()` never found which is misinterpreted as requiring an
argument.

Wording automake

write_http: consistent multi-instance support

This makes the plugin use `<Node>` blocks liks most other write plugins,
while maintaining backwards compatibility with `<URL>` blocks.

It's a follow up to #899, which was merely a fix for the release
branches.

write_http: make callback names context-dependent

This allows multiple destinations to work again (fixes #821), using the
same logic as other write plugins.

The callback name would now be something like:
    `write_http/http://example.com/endpoint`
which is not very nice.

The next step would be to change this plugin to use `<Node>` blocks
like many others, and pass the URL as a parameter inside each instance
block. So I see this patch as the minimum required to let 5.3 and 5.4
users use this plugin with multiple destinations.

Conflicts:
src/write_http.c

write_http: make callback names context-dependent

This allows multiple destinations to work again (fixes #821), using the
same logic as other write plugins.

The callback name would now be something like:
    `write_http/http://example.com/endpoint`
which is not very nice.

The next step would be to change this plugin to use `<Node>` blocks
like many others, and pass the URL as a parameter inside each instance
block. So I see this patch as the minimum required to let 5.3 and 5.4
users use this plugin with multiple destinations.

ensure all curl-based plugins follow HTTP redirects

For the sake of consistency.

add Modbus/RTU support to modbus plugin

This allows access to a local RS-485 serial port
via the modbus plugin by specifying i.e.

  Device "/dev/ttyUSB0"
  Baudrate 38400

in a <Host> block.

For now it assumes 8N1; adding another config option
to support other configurations could be done later.

Lightly tested on my local setup.

Signed-off-by: Eric Sandeen <sandeen@sandeen.net>

RPM specfile: misc improvements related to perl files

Avoids installing some files in two different packages & locations. Also
avoid unnecessary dependencies on -perl and -contrib packages.

Supersedes #896.

Thanks to @faxm0dem for helping work things out !

write_riemann: defaults to use TCP and batching

Fixes #891

RPM specfile: install client tools to a dedicated package

Avoids the package containing the daemon to depend on libcollectdclient.

Fixes #879

RPM specfile: handle new write_log plugin

Merge remote-tracking branch 'origin/pr/835'

perl plugin: plugins are linked with *_LIBADD and LIBS not *_LIBS and LIBS

libperl's ldopts include libaries and therefore should go in LIBS not LDFLAGS

Without this configure fails compile its libperl test program because
-lperl comes before the test program on the compiler command line.

Merge pull request #886 from pyr/feature/write_log

Add a write_log output plugin which logs values.

Do not mention plugins with no options.

mention write_log in the README

change license

python: Properly init the empty string with Python3.

Fix compiler warnings with Python3.
https://github.com/collectd/collectd/issues/505

Add a write_log output plugin which logs values.

I find this to be useful when developping new input plugins,
it allows creating a very simple configuration and combined
with `-f` can be used for a very simple workflow.

I'm proposing this, since I think it could be useful for
people wanting to debug their config on local agents as well.

Merge pull request #870 from ifesdjeen/bugfix/absolute-in-multivalue-dispatch

Change "plugin_dispatch_multivalue" to accept any metric type.

Merge branch 'collectd-5.4'

Merge branch 'collectd-5.3' into collectd-5.4

Merge branch 'collectd-4.10' into collectd-5.3

Fix check for htonll in configure.ac

Move last_poll_data to inside ceph_daemon struct

Also add backup search for last poll data

Merge pull request #847 from mihu/bind_zone_fix

bind-plugin: fix zones iteratation upper limit

types_list: fix carriage return sign

Many thanks to @NsLib for mentioning this !

RPM specfile: avoid packaging some files twice

Fixes #874

Changes to use base types, add comments, etc.

Fix formatting

Add ceph base types

RPM specfile: revert back to sysvinit on EL6

Feedback on ca84b19de about using upstart by default on EL6 was
negative. This init system is not very well integrated on this distro,
so users would get confused.

configure: Fixed check for AM_PATH_LIBCCRYPT

Change "plugin_dispatch_multivalue" to accept any metric type.

Currently, "plugin_dispatch_multivalue" works only with
"gauge_t" metric type. This commit changes it to accept a
"store_type" (one of "DS_TYPE_{GAUGE|COUTNTER|ABSOLUTE|DERIVE}").

collectd.confpod: mention 2 new options to tail plugin

... which were added in #673

collectd.conf example: mention new write_riemann options

collectd.conf example: remove erroneous quotes

Thanks to @ymettier for pointing that out !

Merge branch 'collectd-4.10' into collectd-5.3

Conflicts:
src/libvirt.c

virt plugin: only gather stats for running domains

Conflicts:
src/libvirt.c

Merge pull request #857 from Lupul/patch-1

virt plugin: Fix memory stats to use the same units

For the record, this is the same as 7bf99b36d, which got missed out from 627a050790.

virt plugin: Fix memory stats to use the same units

memory_submit and memory_stats_submit should use same units "Bytes"

- Fixes: #856

Merge pull request #1 from mfournier/ceph

ceph: a couple of small details

ceph: prepend all warning & error messages with plugin name

Merge pull request #853 from vincentbernat/fix/kafka-unavailable-partitions

write_kafka: check for partition availability before selecting one

write_kafka: check for partition availability before selecting one

When a partition is unavailable, sending to it will just lead to a lost
metric. Therefore, after selecting the partition, check if it is
available. If not, select the next one until we tried them all.

A future iteration may use consistent hashing to avoid to double the
work done on a partition when the previous one is unavailable.

ceph: replace erroneous format string in DEBUG statement

Fixes build error on 32bit systems

ceph: correct pod formatting error in manpage

modbus: wrap long lines + indentation/alignment

added man page section for ModbusRegisterType

renamed Function to ModbusRegisterType

added possibility to choose between holding and input registers

bind-plugin: fix zones iteratation upper limit

add missing credits to Michael Schenck

upstart example config: clarify expect stanza ambiguity

redhat initscript: move restart block to a function

This ensures the "condrestart" option will work.