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

Fixed leak and hint

Added suggested fixes

SNMP Agent plugin: Fix coverity issues

Change-Id: Icb74f9978833191a390c952a09695ccb60ff8452
Signed-off-by: Mozejko, MarcinX <marcinx.mozejko@intel.com>

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

RFE: Add plugin name field into plugin context use it to improve error reporting

plugin_mmock.c: Added daemon_log()

common.c: Use plugin-name prefixed error reporting

upddated correct file

Sorry,
committed the patch by mistake. now fixed.

Fixed additional comments

Added "network plugin" prefix to Errors
return -1;  added to sfree

core: Introduced daemon_log() for messages prefixed with plugin name

bind plugin: Switched to use of cf_util_get_boolean() and cf_util_get_string()

configfile.c: Use plugin name from plugin context in error reporting

This commit also removes 'plugin: ' prefix from `plugin_log()`.

Updated codestyle

src/daemon/plugin.[ch]: Add the plugin name to the context.

Issue: #1949

redis plugin: Fix memleak

Reset LDFLAGS before testing for strtok_r

They may contain all kinds of things that make the test fail.
See #2823 for details