fix c_avl_iterator_prev (#2917)

utils_avltree: fix 'c_avl_iterator_prev()' and cover it by tests

Merge pull request #2913 from StarryInternet/sensors-humidity

Add support for humidity in sensors plugin

add humidity for lm_sensors version >= 0x431

Merge pull request #2915 from jplitza/fix-2076

nfs: Fix number of fields for NFSv4

nfs: Fix number of fields for NFSv4

The previous buffer of 64 fields was too short for the current number of
72+2 fields in NFS 4.2, causing the message

  nfs plugin: Unexpected number of fields for NFSv4 server statistics: 62.

This change, originally proposed in [1], fixes that issue in a
forward-compatible manner by reusing the already known number of fields
from the static arrays.

[1]: https://github.com/collectd/collectd/issues/2076#issuecomment-350466628

add humidity for lm_sensors version >= 0x402

add support for humidity in sensors plugin

Fix Solaris builds

Trying to fix error:

./build.sh: syntax error at line 78: `TOP_SRCDIR=$' unexpected

Merge pull request #2864 from jimis/libxml2_configure_fix

Add --with-libxml2 option to configure.

Merge pull request #2907 from Stackdriver/pkg-fix

Fix warning that pkgdatadir and pkglibdir were previously defined.

Add --with-libxml2 option to configure.

It's now valid to disable building with libxml2, using --without-libxml2. It's
also possible to set a custom prefix --with-libxml2=/custom/path.

Merge pull request #2896 from mlichvar/chrony

chrony: Ignore late responses

chrony: Ignore late responses

If a response from chronyd was not received before the timeout (e.g. on a
busy system), it would wait in the receive queue and be processed as a
response of the next request. It would fail the sequence check and cause
the plugin to return with an error. The latest response would wait in
the receive queue and this would repeat forever.

Flush the receive queue before sending a new batch of requests to
prevent this situation from happening.

Fix warning that pkgdatadir and pkglibdir were previously defined.

Merge pull request #2904 from rpv-tomsk/redis-connect-unixsock

redis plugin: Added option for connecting via UNIX socket

redis plugin: Added option for connecting via UNIX socket

redis plugin: Fix coredump when stopping Collectd with inaccessible Redis server

Minor codestyle fix

Merge pull request #2898 from romanek-adam/bugfix/memory-leak-in-plugin-dispatch-multivalue

daemon: make plugin_dispatch_multivalue() obey write queue limits

daemon: make plugin_dispatch_multivalue() obey write queue limits (#2897)

dbi plugin: Removed excessive error reporting

Latest libdbi code sets BADIDX error when no more rows is
in 'dbi_result_next_row()' (similar to 'dbi_result_prev_row()'
and to match library documentation).

There is confusing errors in Collectd logs due to this change.
Added a 'dbi_result_has_next_row()' check to solve.

Closes: #2894

Merge pull request #2882 from Stackdriver/gnulib

Add GNULIB_DIR to LDFLAGS in configure.ac on Windows

Merge pull request #2885 from Stackdriver/plugins

Enable additional plugins on Windows.

Enable additional plugins on Windows.

Add GNULIB_DIR to LDFLAGS in configure.ac on Windows

Base port to Windows (#2810)

* Update configure.ac / Makefile.am to build for Windows using Cygwin.
* Update build.sh to build for Windows.
* Base port of the daemon.
* Include gnulib as a submodule.

configure.ac: Split libxml2 and libvirt mix

Merge pull request #2878 from nhuff/freebsd_pending_io

Report number of in progess disk IO requests on FreeBSD.

Fix formating

Report number of in progess disk IO requests on FreeBSD.

Removed parentheses around return arguments

amqp1: clang-format

Merge pull request #2876 from abays/fix_amqp1_typo

Fix typo in amqp1 JSON format error message

Fix typo in amqp1 JSON format error message

processes: Fix conversion of 'delay_rate' values. Now they really have seconds per second unit.

utils_latency: Use plugin name from context to do error reporting

utils_tail: Added plugin name to error reporting

tail_csv: Fix typo in Synopsis

Merge branch 'collectd-5.8' into collectd-master

Merge pull request #2854 from rpv-tomsk/collectd-master

routeros: Use MAC-address when Radio-name is missing

Fix parsing option for avoiding making BaseDir

Parsing is broken since:
386ecab4 Reading configuration file is now done outside main().
So after mentioned commit we've got:

collectd: invalid option -- 'B'

This commit fixes this.

Merge pull request #2856 from manio/master

Fix parsing option for avoiding making BaseDir

Fix parsing option for avoiding making BaseDir

Parsing is broken since:
386ecab4 Reading configuration file is now done outside main().
So after mentioned commit we've got:

collectd: invalid option -- 'B'

This commit fixes this.

routeros: Use MAC-address when Radio-name is missing

routeros: Fix codestyle of latest change

Merge branch 'collectd-5.8' into master

Merge pull request #2851 from rpv-tomsk/external-manio-master

routeros: Add support for temperature and voltage data, fixed typo in 'sectors_written' metric name

routeros: Fixed typo in 'sectors_written' metric name

routeros: Use 'voltage' and 'temperature' as reported types

routeros: Fixed check of 'Collect*' options

routeros: Added conditionals around collect_health code

doc: add information about routeros CollectHealth option

routeros: add support for collecting health data

It includes the voltage and temperature data on supported hardware.

Auto-Merge pull request #2850 from rpv-tomsk/collectd-collectd-5.8

Automatically merged due to "Automerge" label

write_prometheus: Added MHD_USE_INTERNAL_POLLING_THREAD flag

Since 0.9.53, libmicrohttpd requires explicit setting of this flag.

Closes: #2849

turbostat plugin: show error details

Merge pull request #2733 from elfiesmelfie/feat_pcie_aer

New plugin to read PCIe errors

pcie_errors: Remove 'pcie_error' from types.db

Plugin does not send metrics, type is not needed to send notifications.

Merge pull request #2631 from DanCech/tagged-carbon

Graphite 1.1+ tag support

Merge pull request #2847 from rpv-tomsk/collectd-master

tree-wide: Use interval value from plugin context, do not set 'vl->interval' in plugins more.

Merge pull request #2845 from rpv-tomsk/tmpfix

redis plugin: Remove `operations_per_second` metric.

write_kafka: Added Graphite tag support

write_graphite: Documented 'UseTags' option

utils_format_graphite: add tests for graphite tag flag

utils_format_graphite: Add graphite tag support

curl plugin: Added 'Interval' option

curl_xml plugin: Added 'Interval' option

core: Added plugin name to (some) error reporting

tree-wide: Use interval value from plugin context

core: Cleanup in vl->interval assignment

core: Set plugin name in context when plugin is autoloaded

core: Set 'interval' value in plugin context to match read callback interval

postgresql: Set correct value to 'interval' query parameter (bugfix)

utils_tail_match: Updated formatting

utils_tail_match: Removed unused variables

utils_tail_match: Fix formatting

Merge pull request #2840 from elfiesmelfie/fix_deprecated_func

dpdk: fix deprecation warning

Merge branch 'master' into fix_deprecated_func

statsd: Improve error reporting

redis plugin: Remove `operations_per_second` metric.

The `operations_per_second` metric removed as duplicate of other metric and as
it has wrong value also. Collectd should not report wrong values.

Metric `operations_per_second` is based on `instantaneous_ops_per_sec` field
from Redis `INFO` output. That field is calculated based on buffer of 16 values,
updated from `stat_numcommands` once per 100ms.
So, `instantaneous_ops_per_sec` value has average only for last 1.6 seconds.
While Collectd poll period is 10 seconds, statistics for 8.4 seconds is lost.

Herewith, Collectd already has correct metric `total_operations`, based on
`total_commands_processed` field taken from same `stat_numcommands` internally
in Redis. Value of this metric should be used instead of removed metric.

Merge pull request #2844 from elfiesmelfie/fix_snmp_agent

SNMP Agent plugin: Fix coverity scan issues #2814

dpdk: fix clang issue for deprecated function patch

Signed-off-by: Kevin Laatz <kevin.laatz@intel.com>

cleanup: Remove excessive error reporting

These messages already reported in parse_value()

Auto-Merge pull request #2838 from rpv-tomsk/collectd-master

Automatically merged due to "Automerge" label

Merge pull request #2831 from elieyal/patch-3

network plugin: Add BindAddress option

Merge pull request #2834 from rpv-tomsk/plugin-context-master-2

notify_email: Include all notification parameters into email

redis plugin: Report keyspace hitratio

Merge pull request #2837 from abays/fix-collectd-tg-dtime

Use CLOCK_REALTIME for collectd-tg times (fixes issue 2219)

Updated change from debug to error

As suggest by @rpv-tomsk

dpdk: fix deprecation warning

This fixes a warning produced by a change introduced in the latest DPDK
relase caused by a deprecation notice.

Signed-off-by: Kevin Laatz <kevin.laatz@intel.com>

Use CLOCK_REALTIME for collectd-tg times (fixes issue 2219)

revert back to DEBUG

will change this later, it's not as simple as it looks.

chenged debug to error message

notify_email: Include all notification parameters into email

Auto-Merge pull request #2833 from rpv-tomsk/plugin-context-master-2

Automatically merged due to "Automerge" label

cleanup: cf_util_get* instead of local copy in plugins, prefixed error reporting

Closes: #2011 (superseedes)
Issue: #1949
Issue: #2799
Issue: #2821

removed some detaild in the BindAddress documentation

Removed some details.

snmp_agent: Try to fix tests

This adressed to fix Solaris 10 builds.

Issue: #2814

 network plugin: Added documentation to BindAddress option

Added the BindAddress explanation in the network plugin section on how to use the BindAddress.

Fixed all comments by @rubenk