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

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

Automatically merged due to "Automerge" label

Fixed all comments

Merge pull request #2819 from rpv-tomsk/snmp-fix-2291-1

snmp plugin: HostOID, FilterOID and other options added

snmp_agent: Try to fix tests

This adressed to fix Solaris 10 builds.

Closes: #2814

snmp_agent: Try to fix tests

This adressed to fix Solaris 10 builds.

Closes: #2814

snmp_agent: Remove structure initialization

This adressed to fix Solaris 10 builds.

Closes: #2814

redis plugin: Added option ReportCpuUsage for reported CPU consumption

snmp_agent: Replace strndup() with internal implementation

This adressed to fix Solaris 10 builds.

Closes: #2814

redis plugin: Simplified commands statistics parser

redis plugin: Added commands statistics reporting

redis plugin: report CPU usage

Add Bind Address option

This change allows to Bind Address to an IP address instead of the binding to the Interface Name in the server section.
Example:
<Server "239.192.74.66" "25826">
                SecurityLevel Encrypt
                Username "user"
                Password "secret"
                BindAddress "10.9.25.59"
                ResolveInterval 14400
        </Server>

Credit to: Ofir Hermesh

redis plugin: Reworked to use plugin_register_complex_read()

redis plugin: Implemented persistent connections

write_redis plugin: Fixed bug of Timeout option

Documentation states what Timeout is set in milliseconds.
Code was fixed to match documentation.

redis plugin: Report query errors