logparser plugin: move utils_* to src/utils

move utils_* to src/utils
minor doc fixes

Change-Id: I287dff4639b2fa59418cac3e16093b240119f945
Signed-off-by: Kamil Wiatrowski <kamilx.wiatrowski@intel.com>

Fix Klocwork issues

Change-Id: I9189751dd8e29047488d2f9634cb0856333436bf
Signed-off-by: Adrian Boczkowski <adrianx.boczkowski@intel.com>

logparser plugin: Add unit tests

Change-Id: I193aa776d6fbc1e31690db96d85143707823fbe1
Signed-off-by: Mozejko, MarcinX <marcinx.mozejko@intel.com>

logparser plugin: Plugin used to parse log files using regexes

utils_tail, utils_tail_match: Change _Bool to bool.
utils_message_parser: Correct struct and type names, cleanup, unit
tests.

Change-Id: I40f3336e274f01791e7c171db5be2f21a6c9217c
Signed-off-by: Mozejko, MarcinX <marcinx.mozejko@intel.com>

pcie_errors plugin: possibility only to filter messages from log file.

Add new possible value for submatch index: -1. For this value do not
add the matched string to results. It can be used to only filter
the required messages from syslog without adding the match to result.

Change-Id: I93d9595be77427c135cc09baac901cd919d6c27e
Signed-off-by: Kamil Wiatrowski <kamilx.wiatrowski@intel.com>

utils_message_parser: added message buffer shrinkage

Large message buffer memory allocation that was needed
for handling MCE message flood has significant
performance impact on collectd CPU usage.
This fix adds logic for decreasing message buffer size to
default value (64) if there's nothing to parse.

Change-Id: Ibbaef211e1fe24d53663d717a1899ec719017b2b
Signed-off-by: Krzysztof Matczak <krzysztofx.matczak@intel.com>

message log parser util: ignore patterns after message is completed

Extend condition to ignore patterns before message has started
and after it is completed.

Change-Id: I2ea7fcc8c11bf3f41a21aa08b014faf6074d2234
Signed-off-by: Kamil Wiatrowski <kamilx.wiatrowski@intel.com>

Utility for filtering and parsing log messages

This utility simplifies filtering and parsing specific messages
from given log file. It tracks log file using utils_tail_match.h API.
Main feature is automatic messages assembly based on set of user provided
regular expressions containing distinguished expressions for recognizing
start and the end of the message. That makes assembling single message
from multiple log lines quite easy. Utility supports multiple parsing jobs
and also user defined message part validation flags (mandatory or not).

Added fixes for memory leak in utils_match.c and for debug mode
segfault in src/utils_tail_match.c

Modification of utils_tail and utils_tail_match API with adding option
for reading file from the beginning. Minor modifications in this API
calls in tail and tail_csv plugins.

Change-Id: I6865833c8d5403294124187aa7f93cc957004a03
Signed-off-by: Krzysztof Matczak <krzysztofx.matczak@intel.com>

Merge pull request #3340 from ryan-mccabe/pr2365-turbostat-parsing

turbostat: Fix parsing warnings

turbostat: Fix parsing warnings

Fix warnings generated by the turbostat plugin due to trailing
characters while picking out the first number in a range.

ChangeLog: turbostat plugin: Fix parsing warnings.
PR: #2365
Signed-off-by: Ryan McCabe <rmccabe@redhat.com>

Merge pull request #3269 from bplessis/feature-add-snmpbulk

snmp plugin: add support for SNMP Bulk Transfer

Merge pull request #3283 from zebity/master

Consistent returns from ssnprintf/__ssnprintf wrappers, add ssnprintf2 for optional error handling.

Merge pull request #3331 from jkohen/patch-1

Create internal library for utils_llist

Create internal library for utils_llist

We plan to use this library in the write_gcm plugin in the https://github.com/Stackdriver/collectd fork. I hope it's OK with you to make this change upstream before we are ready to contribute the Stackdriver plug-in into this codebase.

Merge pull request #3328 from efuss/fix-3312

Fix libupsclient detection

Merge pull request #3329 from efuss/fix-3311

Fix libtokyotyrant handling

Fix libtokyotyrant handling

In libtokyotyrant handling, set with_libtokyotyrant_ldflags to "-L$withval/lib", not "-L$withval/include".

Fix libupsclient detection

Check libupsclient $withval for being a regular file in addition to being executable/searchable. The value may be a directory name.

Merge pull request #3310 from yousong/mem

network,exec: fix memory issue

Merge pull request #3325 from ffontaine/master

configure.ac: fix protobuf/util/time_util.h detection

configure.ac: fix protobuf/util/time_util.h detection

Add -std=c++11 when checking gor protobuf/util/time_util.h otherwise
test will fail on:

configure:104495: checking google/protobuf/util/time_util.h usability
configure:104495: /home/br-user/autobuild/run/instance-2/output/host/bin/mips-linux-gnu-g++ -c -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64  -Os  -D_FORTIFY_SOURCE=1  -pthread  conftest.cpp >&5
In file included from /home/br-user/autobuild/run/instance-2/output/host/opt/ext-toolchain/mips-linux-gnu/include/c++/5.3.0/atomic:38:0,
                 from /home/br-user/autobuild/run/instance-2/output/host/mips-buildroot-linux-gnu/sysroot/usr/include/google/protobuf/io/coded_stream.h:113,
                 from /home/br-user/autobuild/run/instance-2/output/host/mips-buildroot-linux-gnu/sysroot/usr/include/google/protobuf/duration.pb.h:23,
                 from /home/br-user/autobuild/run/instance-2/output/host/mips-buildroot-linux-gnu/sysroot/usr/include/google/protobuf/util/time_util.h:52,
                 from conftest.cpp:188:
/home/br-user/autobuild/run/instance-2/output/host/opt/ext-toolchain/mips-linux-gnu/include/c++/5.3.0/bits/c++0x_warning.h:32:2: error: #error This file requires compiler and library support for the ISO C++ 2011 standard. This support is currently experimental, and must be enabled with the -std=c++11 or -std=gnu++11 compiler options.

Fixes:
 - http://autobuild.buildroot.net/results/0d486edaf4c51b6438adeec61fe2c55099862acb

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

Fix conflict from remove of ssnprintf2 and align utils/common/common.h

Merge pull request #3309 from rubenk/remove-lvm-plugin

Remove LVM plugin

Auto-Merge pull request #3138 from mkauf/fix_exec_environment

Automatically merged due to "Automerge" label

Merge pull request #3317 from mrunge/check_fixes

Merge check fixes from 5.9 branch to master

Fix warning in test_strunescape

src/utils/common/common_test.c:254:3: warning: ‘strncpy’ output truncated before terminating nul copying 16 bytes from a string of the same length [-Wstringop-truncation]
  254 |   strncpy(buffer, "\\tbackslash end\\", sizeof(buffer));
      |   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  CCLD     test_common

(cherry picked from commit 5bcc3a10cbae9d94ccb10a62218ab26433bbaa67)
(cherry picked from commit 7109b8a27f2d82fbf8c1ae9d58ab6cffbaec9339)

Fix warning in test_escape_string

In function ‘test_escape_string’,
    inlined from ‘main’ at src/utils/common/common_test.c:384:3:
src/utils/common/common_test.c:226:5: warning: ‘strncpy’ specified bound 16 equals destination size [-Wstringop-truncation]
  226 |     strncpy(buffer, cases[i].str, sizeof(buffer));
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

(cherry picked from commit 737ffc08e2d9f3d3dda40a51329881085070863e)
(cherry picked from commit 52e8daca0c0b53c61c31fe9ea058b08e6eb7c2b1)

Fix warning in test_escape_slashes

In function ‘test_escape_slashes’,
    inlined from ‘main’ at src/utils/common/common_test.c:383:3:
src/utils/common/common_test.c:201:5: warning: ‘strncpy’ specified bound 32 equals destination size [-Wstringop-truncation]
  201 |     strncpy(buffer, cases[i].str, sizeof(buffer));
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

(cherry picked from commit a22c4260335616b7847e57ef638e0e29cd8cda05)
(cherry picked from commit d5edf1e82feb80cfe90cfa35501eef3c8dc493e5)

Merge pull request #3308 from rubenk/travis-osx

Travis improvements for MacOS

network: fix data race when accessing sending_sockets

This can happen with more than 1 WriteThreads

exec: free up memory on shutdown

Stop installing libstatgrab

It replaces native functionality

Try to point OSX at where java is

Show config.log for debugging

Try to use java

Install a few deps

Sort output of pkg-config

Remove LVM plugin

liblvm2app has been deprecated

Log which packages pkg-config thinks we have

Travis on MacOS: bump image version

Move the variable declaration closer to use

Fixes from ruben review

Merge pull request #3299 from mrunge/master

Merge stable branch back to master

Merge pull request #3291 from dago/issue3220

Link to libnsl.so if needed for inet_ntop()

Merge tag 'collectd-5.9.2'

Merge pull request #3298 from rubenk/contextswitch-nosysctl-h

Only include <sys/sysctl.h> when needed

Only include <sys/sysctl.h> when needed

This plugin doesn't need sysctl.h on Linux.

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

exec: fix a race condition when setting environment variables

Pass the environment as a parameter to the child process
and don't modify the environment of the parent process.

Follow-up to 0684aea7

Closes #3041

Link to libnsl.so if needed for inet_ntop()

Add changelog to prepare 5.9.2 release

Merge pull request #3247 from jplitza/fix-2843

zfs_arc plugin: Read dbuf_size, dnode_size and bonus_size

zfs_arc: Read dbuf_size, dnode_size and bonus_size

They are the replacement for other_size since ZFS on Linux 0.7.0

Auto-Merge pull request #3286 from mrunge/master

Automatically merged due to "Automerge" label

Merge pull request #3288 from usev6/mysql_wsrepstats_doc

Tweak docs for mysql plugin

Add ssnprintf2 wrapper variiant definition

mysql plugin: Fix typo in documentation

mysql plugin: Tweak docs for WsrepStats option

* Avoid rendering as preformatted text.
* Note that config option defaults to false.

Fix typo in src/java.c

The error message did not contain a 'not'

Closes: https://github.com/collectd/collectd/issues/3285

Merge pull request #2622 from abays/connectivity

Red Hat NFVPE Connectivity Plugin

Merge branch 'master' into connectivity

Merge pull request #3282 from abays/sysevent

Check for sysevent plugin init calloc failures

Add second wrapper ssnprintf2 to allow introduction of error handling on truncation in future

Check for sysevent plugin init calloc failures

Make return result consistent with fixed src/utils/common/common.c - ssnprintf wrapper

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

Change-Id: Idab2c4ffa242a73b651f1b056f85f905e356c5ac

Merge pull request #3280 from mrunge/collectd-5.9

Collectd 5.9

Merge pull request #3276 from chrismcn/upstream_turbostat

Upstream turbostat

Merge pull request #3277 from abays/sysevent

Use sstrncpy instead of strncpy in sysevent plugin

Fix compile time issues

This resolves #3242 and should also resolve #3179.

Signed-off-by: Matthias Runge <mrunge@redhat.com>
(cherry picked from commit 0bbf058d6f9935e6e727cda7db79312281e2f58d)

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

Use sstrncpy instead of strncpy in sysevent plugin

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

Change-Id: Idab2c4ffa242a73b651f1b056f85f905e356c5ac
(cherry picked from commit d27bfc74e606290f191c04cdb4e2acc4d5e8d4a9)

Added ICX Xeon support model number.

Change-Id: I34ef3a2180a86e26d82c046a58e2862f3bc65cf5

Changes for power reporting on SKX and CLX.

Change-Id: I2a6e9faebe616102904bcba4bc51e8e974a56408

Merge pull request #3159 from rpv-tomsk/collectd-usercache

check_uptime: New plugin, based on new cache_event callback.

Bump for ChangeLog re-check

Fix AUTHORS

Remove duplicate author entry

clang formatting, again

Bump CI for connectivity plugin

More styling + ignorelist create check

Handled interrupted socket reads

Use a separate dequeue thread to dispatch notifications

connectivity plugin initial commit

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