More styling + remove superfluous dequeue thread error logic

octo code review changes

Use a separate dequeue thread to dispatch notifications

Add libmnl-devel req for connectivity

Spec file typo fixes

Connectivity RPM specfile

Properly detect which interfaces to monitor

Use ignorelist + malloc fix + conf clarification

monitor all interfaces by default

snake case for iface list + payload simplification

Clarify wanted netlink interface msgs

connectivity notifications

connectivity plugin initial commit

Merge pull request #2624 from abays/sysevent

Red Hat NFVPE Sysevent Plugin

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>

Merge pull request #3237 from ccin2p3/f/temp-fix-ssnprintf

fix ssnprintf wrapper

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

fix https://github.com/collectd/collectd/issues/3232

Change-Id: Idab2c4ffa242a73b651f1b056f85f905e356c5ac

Add Changelog for collectd-5.9.1

(cherry picked from commit 47f8966a2ea076a918d0d1383765621651337881)

Auto-Merge pull request #3229 from mrunge/cppflags_backport

Automatically merged due to "Automerge" label

Merge pull request #3230 from ryan-mccabe/collectd-5.9

Backport fixes for collectd#3225 to collectd-5.9

virt plugin: Remove strdup() from MetadataXPath implementation

Issue: #2805
Issue: #3225

virt: Fix memory leak with libvirt MetadataXPath enabled

Issue: #3225

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

Automatically merged due to "Automerge" label

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

(cherry picked from commit e4c981472890655a5113d5b3bd643f62f19d0139)

Fix typo in src/network.c

(cherry picked from commit 262212d3e5db97743c9ff47897778a84049e4bae)

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 #3202 from mrunge/collectd-5.9

Collectd 5.9

Clang-format a few files to make CI happy

Another round of clang-format

related to backports/cherry-picks.

Run all changed files 68 8.c/1*.h through format.sh web util

(cherry picked from commit 27ca3ca92a757b5c9bb3baa5a29ef352ec70b1c9)

Run all changed files 68 8.c/1 *.h through clang-format agin..

(cherry picked from commit 626d3ab4309cdb069289354c45ca9e2ba9bc9a2f)

Run all changed files 68 8.c/1*.h through clang-format

(cherry picked from commit eada465ff40eb2ecaef17490e5462dab3b8874fe)

Add snprintf wrapper for GCC 8.2/3

(cherry picked from commit abaa1c8a24e8eff5632dd6052b1da5f6535caf19)

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

865f2eb31e8264edd9337bc63ac59f6c2fa3b5a7 breaks locale checks

Issue: #3181
(cherry picked from commit 38fac5d5579b952f9546623f39cdbcae89a6f090)

disk.c: remove unused lines

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

Issue: #3181
(cherry picked from commit d99416997de1c4d9e676960719327a52a364f605)

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

(cherry picked from commit a72ad42076337f3b85db3b06565ae86a83226ab3)

Move Makefile rules for pid_test inside conditional for code

(cherry picked from commit 45b8c056d89f4b029e6017035ce6a4a37a36d260)

Add missing definitions for libnetsnmpagent

(cherry picked from commit 3b05d9f7b7b46e0c4fee296ad2d6f16ecc82173e)

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

dns plugin: Do not use headers from glibc

glibc has removed __NAMESER and __BIND symbols from their codebase.

There is no benefits of use defines from glibc, so replaced them by numbers.

Closes: #3145
(cherry picked from commit faa20ef796abc91656c5c583970db9a9f4daf911)

Fix return code introduced by #3182

Change-Id: I0972f74f3fff05cf29fa9b0be383f0b0df1e6d03
Fixes: #3200
(cherry picked from commit 835a58c09ff7afd44627915c9274b16c5df0fb50)

Use pid_test only when compiling the plugin that uses it

(cherry picked from commit a7401cc81d05bf4f230aaf35e4109a23b60fc546)

clang-format

(cherry picked from commit 4af2afc84a90fe37e4988fca08ea79a6343c6b94)

better user feedback

Change-Id: Iae7c3208024372485fd0901898cbe2e178610082
(cherry picked from commit d75591c9762f93d8656846fd2796fdb775b1c69f)

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
(cherry picked from commit a07f44c9cd5828a84545dcfe22544bfed2ea8812)

Use GCC-specific flags only when compiling with GCC

(cherry picked from commit 9317d0734808f56a5d181ff2864332705fcb87e0)

Make sure files in this changeset are formatted properly

(cherry picked from commit 9315ccb08ebd29f0b79426853c2e7adf448c26e9)

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

(cherry picked from commit 54d4f889eb72249824ef4ee6b155b0e0fb858be2)

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
(cherry picked from commit 42a30efe33c4d16276589e0dd1f9e135b38e054d)

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>
(cherry picked from commit d4f53887527777296b0982af60f4c0fecdb2f0d9)

fix comment

Change-Id: I655dd0b3cb82053f7cc50dec149d4f7cc25ee18b
(cherry picked from commit 2d485cf961eb65d4a55c5c64aa40fdbcca8f41f1)

fix deps

Change-Id: I8199655fcd22082a2f3cfb0e5ab7db289df1a5e7
(cherry picked from commit 9e5b363dda7306b628c1bf91c349ed51f11dbae6)

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
(cherry picked from commit 09abfaaeb71957c6039387ad3f5d82803c14ddda)