use cdtime + snprintf cleanup + other styling/cleanup

More styling + ignorelist create check

Handled interrupted socket reads

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 remote-tracking branch 'abays/procevent'

More clang formatting for unrelated plugin

Another string formatting fix attempt

Debian string formatting fix

Fix merge conflicts

clang formatting

Align with new util library locations

Styling + optimizations

Handled interrupted socket reads

More styling + remove superfluous dequeue thread error logic

Additional mutex + styling changes

Use a separate dequeue thread to dispatch notifications

Remove types.db entry and change RegexProcess conf option name

Use long type for most PID values

clang formatting

Drain netlink socket before sleeping, then filter repeated process start msgs

Procevent RPM specfile

Check ignorelist during init

Fix mutex lock in read_event

Use ignorelist_t and fix leaks

payload logic clean-up + leak fixes

Clarify wanted netlink proc msgs

Procevent plugin notifications

check fscanf return value in process_check

procevent 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>

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