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.

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

redis plugin: Update documentation

redis plugin: Change text of some error messages

redis plugin: Check for / report connection errors

redis plugin: Fixed bug of Timeout option

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

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

Automatically merged due to "Automerge" label

rrdtool plugin: Added more error reporting

Issue: #2823

Auto-Merge pull request #2822 from rpv-tomsk/issue-976

Automatically merged due to "Automerge" label

ntpd plugin: Added new refclock_names

ntpd plugin: Don't treat normal peers as refclocks.

Don't try to interpret the srcadr field as a refclock
identifier unless it actually is a refclock identifier.
Now Collectd logs all stats for peers rather than skipping
delay for most of them and offset for a few.

Closes: #976

snmp plugin: Updated documentation for 'Ignore' option

snmp plugin: polished a code

snmp plugin: Removed newly-added 'suffix skipped' notice

snmp plugin: FilterOID option implementation

Added new options `FilterOID`, `FilterValues` and `FilterIgnoreSelected`.
These allows to ignore parts of the MIB by its values.

Closes: #2637

snmp plugin: HostOID option implementation

+ Allow to use `TypeInstanceOID` and `PluginInstanceOID` together.
+ Added `TypeInstancePrefix`, `PluginInstancePrefix` and `HostPrefix` options.
* Option `InstancePrefix` marked as deprecated.
+ Added documentation for new feature.

Closes: #174

snmp plugin: Replace reserved chars in hostname value

snmp plugin: added NOTICE when suffix (row of values) is skipped

snmp plugin: Implemented HostnameOID option

snmp plugin: Refactored filling cells of table

This also fixes a bug when `Ignore` option was ignored for numeric Instance OIDS.

snmp plugin: Rename types and fields to more common names

snmp plugin: codestyle

snmp plugin: Pass variable to csnmp_instance_list_add()

Instead of scanning `res->variables` use directly-passed variable.

snmp plugin: Added types into list of requested snmp OIDS

Issue: #174

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

snmp plugin: Implemented new configuration options

snmp plugin: Implemented new configuration options

Added new options `PluginInstance`, `TypeInstance`, `TypeInstanceOID` and `PluginInstanceOID`.
These allows flexible configuration of reported metrics.

Existing `Instance` option marked as deprecated.

Closes: #2636

docs: Fix typo

snmp plugin: Allow to set plugin value in reported metrics

Auto-Merge pull request #2813 from rpv-tomsk/master

Automatically merged due to "Automerge" label

config parser: Improved error reporting on global options

Some functions could return error code without any messages.

perl plugin: Restore missing include of "collectd.h"

Restore include which was removed in fa36b45bfafec663e49639d40f4671e018e7ee10 (#2769).
This adressed to solve compilation issue on Solaris platform.

Merge pull request #2702 from elfiesmelfie/feat_snmp_agent

snmp_agent: new features

SNMP Agent plugin: Fix minor issues

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

Merge pull request #2804 from rubenk/remove-poison

Stop poisoning function in debug mode

Merge pull request #2808 from elfiesmelfie/fix_virt_shadow_warning

src/virt.c: Fix one 'shadow type' warning