clang-format

Addapt conf files

VSM only apply for Varnish 4

Remove old varnish and support for varnish 6

Factor out read_text_file_contents to ensure that data read from text files is a valid C string.

Updated a few metrics names and descriptions to be more intuitive.

Signed-off-by: TG, Hari <hari.tg@intel.com>

Merge pull request #3353 from mrunge/master

Remove grpc from Mac builds for now

Merge pull request #3352 from ryan-mccabe/leak-fixes

Leak fixes

Remove grpc from Mac builds for now

Merge pull request #3344 from Self-Perfection/patch-2

Fix linking with Python 3.8

updates based on review comments

Merge pull request #3351 from ccin2p3/f/write_riemann-memleak

Fix memory leak in write_riemann

Fix memory leak in write_riemann

Trivia: this cost a user €1844,28160184475 in electricity (~3TB of RAM
freed)

Changelog: Fix memory leak in write_riemann
Change-Id: If95da8d90c078c19ac0062e3bbb006185dcc441d

network: Add missing freeaddrinfo on error path.

Add a missing freeaddrinfo on an error path to fix a memory leak
in the network code.

Changelog: network: Fix memory leak.
Signed-off-by: Ryan McCabe <rmccabe@redhat.com>

amqp1: Fix leaks on error paths

Fix a few memory leaks on error paths in the amqp1 plugin.

ChangeLog: amqp1 plugin: Fix leaks on error paths.
Signed-off-by: Ryan McCabe <rmccabe@redhat.com>

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>

Merge pull request #3349 from rpv-tomsk/master

daemon: Fix regression caused by #3217

daemon: Fix regression caused by #3217

Empty <Plugin> section can be used to autoload plugin,
which does not supports configuration callback.

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>

Rewrite to use only one config array

Fix linking with Python 3.8

Since Python 3.8 --embed flag needs to be provided to python-config to embed python.
Reference: https://bugs.python.org/issue36721

This was partially addressed in #3170 but only fixed building and not linking with python.

Address some clang-format issues

Re-format some lines to appease clang-format.

clang-format also wants to change the indentation of some #endif comments I didn't introduce and re-order #include's into a form that /does not compile/.

ChangeLog: Add recent bugfix merges …

… for the Turbostat and GPU NVidia plugins.

Merge branch 'collectd-5.9' into collectd-5.10

Merge branch 'collectd-5.8' into collectd-5.9

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>
Signed-off-by: Florian Forster <octo@collectd.org>

Format BSD constrained sysctl usage

Avoid using sysctl on linux & allow for OpenBSD case

Avoid using sysctl on linux

Avoid using sysctl on linux

Avoid using sysctl on linux

Avoid using sysctl on linux

Merge branch 'master' of https://github.com/collectd/collectd

Avoid using sysctl on linux

Avoid using sysctl on linux

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>

Reformatted code.

Update utils_cache.h

Expose meta_data_toc function.

This is useful for traversing all the meta data elements in a value list, since there is no element to look up the metadata object.

For example usage, see https://github.com/Stackdriver/collectd/pull/166

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

Merge pull request #3332 from rubenk/cuda-3320

Fix compilation with libcuda

Implement NFS plugin for NetBSD

Revert accidental commit to master branch

Implement NFS plugin for NetBSD

Fix compilation with libcuda

Fixes #3320

buddyinfo: remove redundant sfree

Signed-off-by: Asaf Kahlon <asafka7@gmail.com>

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

buddyinfo: fix review comments

Signed-off-by: Asaf Kahlon <asafka7@gmail.com>

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>

buddyinfo: fix review comments

Signed-off-by: Asaf Kahlon <asafka7@gmail.com>

Merge pull request #3323 from rubenk/cuda-fix-build

gpu_nvidia plugin: fix build

gpu_nvidea plugin: fix build

Fixes #3320

Merge pull request #3321 from ccin2p3/collectd-5.10

fix missing package in specfile

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

fix missing package in specfile

Change-Id: Ic4bbe3ffa60be0f0c2ca78487119ec029a9a4ce0

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

Remove LVM plugin

Merge pull request #3318 from mrunge/collectd-5.10

Changelog and preparation for collectd 5.10

Auto-Merge pull request #3138 from mkauf/fix_exec_environment

Automatically merged due to "Automerge" label

Fix typo

Update the spec file to 5.10.0

Merge branch '5-10-changelog' into collectd-5.10

Prepare 5.10 release

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 #3316 from mrunge/checkfix_collectd59

Merge make check fixes from collectd 5.8 to 5.9

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)

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)

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)

Merge pull request #3307 from rubenk/travis-bionic

Travis: switch to Bionic

Merge pull request #3308 from rubenk/travis-osx

Travis improvements for MacOS

Merge pull request #3306 from rubenk/fix-make-check

Fix make check

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

Disable lvm plugin in Travis for now

In file included from ../../src/lvm.c:29:
/usr/include/lvm2app.h:21:2: error: "liblvm2app is deprecated, use D-Bus API
      instead." [-Werror,-W#warnings]
 ^
1 error generated.
Makefile:7018: recipe for target 'src/lvm_la-lvm.lo' failed