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

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>