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

Fix clang formatting in grpc.cc

Remove duplicate author entry

Fix merge conflicts

Cast cdtime_t to unsigned long long when necessary

Attempt to align with new util library locations

clang formatting, again

Bump CI for connectivity plugin

Styling/optimization cleanup + proper use of cdtime

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 pull request #2624 from abays/sysevent

Red Hat NFVPE Sysevent Plugin

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

Changelog: processes plugin: get # processes running from /proc/stat

processes: update code style

plugin: processes: return error if procs_running can not be retrieved

Although returning '1' is probably right, it's up to the application to
decide. The data structure for number of running processes is an 'int',
so '-1' clearly indicates something's wrong when interpreting the data.

Conflicts:
src/processes.c

plugin: processes: use a const ref string and strlen i.s.o. magic offset

The simple parser is based on 'strstr', whereas the actual data does
include a white-space character to separate the 'id' (procs_running)
from the 'value' (a number). Yet parsing the number can off course only
take place 'after' the white-space. Subsequently, not finding a
'white-space' should be treated as an error condition.

Adding the 'white-space' in const string and defining the offset based
on 'strlen' fixes this properly.

Allow cURL Statistics option in write-http plugin

write-http can optionally configure cURL Statistics as per other cURL based plugins

ChangeLog: write_http plugin: add ability to configure cURL Statistics

fixed code format

New redfish plugin

- Collect data from Redfish interface
- Use libredfish C library

redfish plugin: addressing comments from PR #2926
Add unit tests.
Fix race condition.
Fix segmentation fault.
Fix memory leaks.

ChangeLog: Redfish plugin: Collect sensor data using Redfish protocol.

Signed-off-by: Marcin Mozejko <marcinx.mozejko@intel.com>
Signed-off-by: Adrian Boczkowski <adrianx.boczkowski@intel.com>
Signed-off-by: Zoltan Szabo <zoltan.4.szabo@nokia.com>
Signed-off-by: Michal Kobylinski <michal.kobylinski@intel.com>

code formatting via format.sh

replaced sizeof with NVML_DEVICE_NAME_BUFFER_SIZE (provided with nvml.h)

replaced bit-shift with respective define

added documentation for the two new config options of the nvml plugin

fixed nvml plugin config

Merge pull request #3261 from ffontaine/master

configure.ac: fix typo

types.db: fix ping_droprate range spec

It's always in range 0-1 since the initial support for the data source
in commit 9cc8e54618fe3f8e7af879fcc9e04d71125456fd

write_http no longer writes http response to stdout

use GPU index as plugin instance

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>

fix number of running processes

scanning /proc/*/stat AND computing other process stats takes too much
time. Consequently, the number of running processes based on the occurences
of 'R' as character indicating the running state is typically zero. Due
to processes are actually changing state during the evaluation of it's
stat(s).

The 'procs_running' number in /proc/stat on the other hand is more
accurate, and can be retrieved in one single 'read' call.

In the unlikely case that procs_running can not be retrieved from
/proc/stat, (read interrupted prior to reading up to 'procs_running'
the value '1' assumed, since this plugin is at least being executed.

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

match_regex: Accept all data types of meta data

match_regex currently accepts only string type of meta data, but
there is no reason to reject other data types since they can be
easily stringified by meta_data_as_string().
Accepting all data type will provide more benefit, for example
the following config can distinguish that data is local or remote,
the previous match_regex outputs many error logs for this config
though:

  <Match "regex">
    MetaData "network:received" "true"
  </Match>

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

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