Move the variable declaration closer to use

Fixes from ruben review

Rephrased and simplified option documentation

Add a warning when trying to activate bulktransferts on an SNMP v1 host -- fix formating

Fix typo and add a new 'highend switch'  example that support bulk

Fix typos and try to rephrase description

Add a warning when trying to activate bulktransferts on an SNMP v1 host

Remove extraneous debug information that trigger issue with size_t variable size

name error in the lately added default value setting for bulk_size

Use correct coding style

Add support for SNMP Bulk Transfert

ChangeLog: snmp plugin: support for bulk transfert has been added.

As per issue #2495 we are experiencing so long collect time for switch with high number of port so i took a try at that.

This is the first attempt, it's working but i may not have seen every failure opportunities, i tried to be the less disruptive as possible.

This mostly modify the csnmp_read_table to switch over to 'GETBULK' PDUs if conditions are ok, and handle the extra return values with the same loop.

Comments are welcome

Merge pull request #3261 from ffontaine/master

configure.ac: fix typo

configure.ac: fix typo

Remove double "without" added by
commit b7818712d46e9fbac7bd9dfe93dca936c101d680

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Merge pull request #3241 from ffontaine/master

configure.ac: fix activation of snmp_agent

Merge pull request #3252 from rubenk/lgtm

Fix a few issues found with LGTM

treewide: fix a few missing include guards

found with LGTM

browser.js: declare a few vars used as locals

netlink.c: remove always true comparison

Earlier in the function we check if fields_num >= 1

common.c: remove always true comparison

ntohs() returns an uint16_t, which is always less than 65535

collection3: remove unused variable

collectd_network.py: remove unused import

Merge pull request #3246 from jplitza/zfsarc-skip-lines

zfs_arc plugin: Fix skipping of leading lines

Merge pull request #3239 from usev6/patch-1

Fix indentation of some closing curlies

zfs_arc: Fix skipping of leading lines

Probably due to a merge gone wrong, the first two lines were skipped
twice, skipping four lines in total and thus missing the "hits" and
"misses" counts.

Merge pull request #3245 from mrunge/rdt_fix

Fix compile time issues

RDT-compile fix

include feedback from review, make the code easier.

Signed-off-by: Matthias Runge <mrunge@redhat.com>

Fix compile time issues

This resolves #3242 and should also resolve #3179.

Signed-off-by: Matthias Runge <mrunge@redhat.com>

configure.ac: fix activation of snmp_agent

If netsnmp library has been built without mib loading support (through
--disable-mib-loading), build of snmp_agent fails on:

src/snmp_agent.c: In function 'snmp_agent_get_asn_type':
src/snmp_agent.c:160:47: error: dereferencing pointer to incomplete type 'struct tree'
   return (node != NULL) ? mib_to_asn_type(node->type) : 0;

struct tree is defined in net-snmp/libray/parse.h which is included by
net-snmp/mib_api.h only if NETSNMP_DISABLE_MIB_LOADING is not set.

To fix this error, check for get_tree function in netsnmp library in
addition to init_agent in netsnmpagent library

Fixes:
 - http://autobuild.buildroot.org/results/6d7ac28f154f83208f949c62d28411855f1817f8

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Fix indentation of some closing curlies

Merge pull request #3238 from ccin2p3/f/syslog-debug-level

Don't fail if syslog loglevel doesn't match

Don't fail if syslog loglevel doesn't match

This makes the log message more consistent with the behaviour,
and also is consistent with pre-
3b9c7b21b4ddfcf59a0147bc9e91e8889ca78d56 behaviour

Change-Id: I0e8aa2f5c44cb5b4142001954f4544c4157125c9
Fixes: #3236

Auto-Merge pull request #3228 from rpv-tomsk/issue-3215

Automatically merged due to "Automerge" label

Merge pull request #3207 from dago/cppflagsbugs

Fix bug that leads to CPPFLAGS gets overridden with CFLAGS when libxm…

Fix formatting in src/network.c

Fix typo in src/network.c

virt plugin: Remove strdup() from MetadataXPath implementation

Issue: #2805
Issue: #3225

virt: Fix memory leak with libvirt MetadataXPath enabled

Issue: #3225

Merge pull request #3217 from rpv-tomsk/issue-3215

daemon: Check if plugin actually loaded before reporting configuration issues

Merge pull request #3224 from mrunge/codeowners

Move trusted-contributors down the list

Move trusted-contributors down the list

since the last matching pattern takes the most precedence.

Merge pull request #3214 from rpv-tomsk/issue-3181

Issue 3181

daemon: Check if plugin actually loaded before reporting configuration issues

Issue: #3215

configure.ac: Removed unused checks for functions

daemon: Recover setlocale() call in src/daemon/collectd.c do_init()

865f2eb31e8264edd9337bc63ac59f6c2fa3b5a7 breaks locale checks

Issue: #3181

disk.c: remove unused lines

limits.h check removed in 865f2eb31e8264edd9337bc63ac59f6c2fa3b5a7
limits.h included in src/daemon/collectd.h unconditionally

Issue: #3181

Merge pull request #3153 from zebity/master

Add snprintf wrapper to avoid "Truncation Warnings" with GCC 8.2/3 compiler

Run contrib/format.sh across entire *.h/*.c source tree (2nd of 2)

Run contrib/format.sh across entire *.h/*.c source tree (1st of 2)

Merge pull request #3072 from octo/ff/network_test

Add a unit test for the network plugin.

Merge remote-tracking branch 'upstream/master'

run all updated *.h *.c through clang-format again

Merge upstream, address conflict in src/utils/cmds/cmds_test.c

Merge upstream, address conflict in src/utils/cmds/cmds_test.c

Auto-Merge pull request #3206 from dago/pid_test

Automatically merged due to "Automerge" label

Fix bug that leads to CPPFLAGS gets overridden with CFLAGS when libxxms is enabled

Merge branch 'master' into master

Move Makefile rules for pid_test inside conditional for code

Auto-Merge pull request #3205 from rpv-tomsk/3112-powerdns

Automatically merged due to "Automerge" label

powerdns: Added 'unreachables' reporting

Issue: #3112

powerdns: Fix formatting, broken by clang-format

Auto-Merge pull request #3203 from dago/netsnmpagent

Automatically merged due to "Automerge" label

Add missing definitions for libnetsnmpagent

Merge pull request #3201 from ccin2p3/f/fix-syslog-rc

Fix return code introduced by #3182

Fix return code introduced by #3182

Change-Id: I0972f74f3fff05cf29fa9b0be383f0b0df1e6d03
Fixes: #3200

Merge pull request #3197 from dago/pid_test

Use test_utils_proc_pids only when compiling the plugin that uses it

Use pid_test only when compiling the plugin that uses it

Merge pull request #2725 from RafaelMarinheiro/fix_grpc_query_values

utils_cache: fix pointer reference and meta_data ingestion

Merge pull request #3178 from ashie/fix-match-regex-for-empty-meta

regex match: Fix unexpected match with empty meta data

Merge pull request #3195 from dago/gccflags

Use GCC-specific flags only when compiling with GCC

Merge pull request #3160 from rpv-tomsk/collectd-uptime-memcached

memcached plugin: Report uptime metric

Auto-Merge pull request #3193 from dago/fileoffsetbits

Automatically merged due to "Automerge" label

Merge pull request #3156 from rpv-tomsk/dns

dns plugin: Do not use headers from glibc

Merge pull request #3182 from ccin2p3/f/config-return-codes

Fix return value or loglevel for several plugins

clang-format

better user feedback

Change-Id: Iae7c3208024372485fd0901898cbe2e178610082

Fix return value or loglevel for several plugins

Some plugins' RC was != 0 and started to fail due to
stricter verification introduced by 3b9c7b2.
This commit fixes those return values.
For some plugins, fix verbosity of error message: non-zero rc
should be error, not warning.

Change-Id: I9a3f1f80e266858b6744fd9d9d99b352b8d94306

Change-Id: Ibf6ebc6cdc93c6e105d488e4a131dcb6e8eea19b

Change-Id: I35bac15fa0a89b068575739ac1cff0115c9d3a40

s

Change-Id: I992002c56763fbdea5347e5b6e176cc86f5a08ce

Use GCC-specific flags only when compiling with GCC

Make sure files in this changeset are formatted properly

Add standard include early or _FILE_OFFSET_BITS will have definition mismatches on Solaris

be more verbose when plugin config cb is failing

3b9c7b2 introduced stricter RC check.
This commit provides user feedback when config callback is failing.

Change-Id: Ia9c13048e95559b5be84477fc1602ff418a1df37

Merge pull request #3175 from ccin2p3/f/collectd-redhat-spec-5.9.0

redhat spec: fix build due to new upstream plugins

regex match: Fix unexpected match with empty meta data

If a user set

  MetaData "foo" "bar"

and a metric does not have meta data (vl->meta == NULL), it causes
unexpected match.

Existance of MetaData config should be checked before meta data in
a metric.

See also: #1930

Signed-off-by: Takuro Ashie <ashie@clear-code.com>

fix comment

Change-Id: I655dd0b3cb82053f7cc50dec149d4f7cc25ee18b

fix deps

Change-Id: I8199655fcd22082a2f3cfb0e5ab7db289df1a5e7

New upstream plugins

* nvidia_gpu
* write_stackdriver
* pcie_errors

The first 2 are disabled, as I don't know what lib they require
(configure fails)

Change-Id: I56456b418edd3defb3c5620a5ad58f527951309c

Merge pull request #3170 from stratakis/py38

Add Python 3.8 compatibility

Add Python 3.8 compatibility

From Python 3.8 and onwards C extensions are no longer
linked to libpython so in order to embed python within
an application the --embed flag needs to be added to
python3-config --libs. A fall back is provided as the
command will error out on previous python versions.

References: https://bugs.python.org/issue36721

Merge pull request #3167 from ashie/remove-needless-tab

Makefile.am: Remove a needless tab

Makefile.am: Remove a needless tab

Merge pull request #3163 from rpv-tomsk/virt-1

virt plugin: improvements

Merge remote-tracking branch 'upstream/master'

virt plugin: Use virGetLastError() instead of virConnGetLastError()

libvirt documentation explicitly points us to use virGetLastError():

 In a multi-threaded application, always use the global
 virGetLastError() API which is backed by thread local storage.

virt plugin: minor updates after review

Merge pull request #3158 from smortex/riemann-meta-data

Copy MetaData to Riemann events in write_riemann

virt plugin: Added connection state check via virConnectIsAlive()

Before this fix, if libvirt daemon was stopped, Collectd starts to spam logs with lot of "failed" messages.

With this fix it correctly re-establishes connection.

virt plugin: Added timeout to event_loop thread

As per virEventRunDefaultImpl() documentation, this function will block
forever if there are no registered event handlers.

This leads to Collectd is unable to correctly stop event_loop thread
if libvirtd was restarted.

Added empty timeout callback to fix this.

virt plugin: Disable ExtraStats selectors when unsupported by libvirt

When function is not supported by driver, log filled by continuous messages like:

libvirt: Domain Config error : this function is not supported by the connection driver: virDomainGetDiskErrors

libvirt API allows us to detect such cases and disable unsupported calls.

virt plugin: Fixed typos and inconsistencies

virt plugin: Do not request cpu maps when not required

virt plugin: Added ExtraStats selector 'vcpu'

This allows to disable virDomainGetVcpus calls when hypervisor/driver
does not supports that function.

Closes: #2615