Travis: switch to Bionic

clang-format

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

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));
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

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));
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

buddyinfo: fix copyright authors

The names written originally were copied from another plugin blindly.
Remove them and add myself.

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

buddyinfo plugin: new plugin for memory fragmentation info

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

dpdk_telemetry plugin: refactor clean up of socket close

Instead of closing each socket during failures, just call
clean up function to be more consistent across the file.

Signed-off-by: Reshma Pattan <reshma.pattan@intel.com>

dpdk_telemetry plugin: move read loop reg to module register

Move read loop callback registration to module_register().

Signed-off-by: Reshma Pattan <reshma.pattan@intel.com>

dpdk_telemetry plugin: fix some logs to be debug

Fix some of the logs level to be debug instead of info.

Signed-off-by: Reshma Pattan <reshma.pattan@intel.com>

dpdk_telemetry plugin: use json loads symbol in configure

Use "json_loads" symbol instead of "json_is_object"
in configure.ac .

Signed-off-by: Reshma Pattan <reshma.pattan@intel.com>

dpdk_telemetry plugin: prefix missed lib

Some places missing the lib prefix in
"with_libjansson", so add the missing prefix.

Signed-off-by: Reshma Pattan <reshma.pattan@intel.com>

dpdk_telemetry plugin: place the plugin in sorted order

Place dpdk_telemetry plugin in sorted order in the list.

Signed-off-by: Reshma Pattan <reshma.pattan@intel.com>

dpdk_telemetry plugin: remove while loop from init and read callbacks

Remove socket reconnect logic from init callbacks.
Read callbacks will try to connect to the socket if
not connected.

Signed-off-by: Reshma Pattan <reshma.pattan@intel.com>

dpdk_telemetry plugin: add plugin for DPDK metrics via DPDK Telemetry library

This patch introduces a new plugin for collectd, which consumes DPDK metrics
via the dpdk_telemetry library. The collectd plugin here provides an
easy way to use the DPDK telemetry API to query ethernet device metrics.

The collectd plugin retrieves metrics from a DPDK packet forwarding
application by sending a JSON formatted message via a UNIX domain
socket. The DPDK telemetry component will respond with a JSON formatted
reply, delivering the requested metrics. The dpdk_telemetry collectd
plugin parses the JSON data, and publishes the metric values to collectd
for further use.

This plugin has a dependency on the DPDK Telemetry library, as it must be
"in sync" with the DPDK Telemetry implementation.

Change-Id: If3343aae4c5473f0574465fab0395b7672fa2488
Signed-off-by: Emma Kenny <emma.kenny@intel.com>
Signed-off-by: Brian Archbold <brian.archbold@intel.com>
Signed-off-by: Reshma Pattan <reshma.pattan@intel.com>

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

Set the thread name before detaching the thread

pthread_setname_np() does not work reliably when a thread has been
created in a detached state. Set the thread name first, and then call
pthread_detach() afterwards (if necessary).

This prevents error messages like this one:
set_thread_name("unixsock conn"): No such process

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()

Merge pull request #3290 from mrunge/collectd-5.9

Add changelog to prepare 5.9.2 release

Add changelog to prepare 5.9.2 release

Updated REDME with library upstream URL. Updated AUTHORS.

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

Improved error handling in case of init and read callback, and collection thread failures.
Updated a few error messages.

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

clang formatting

Allow autoload of matches and targets in the same way of plugins.

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

Apply contrib/format.sh

Change logic to use arrays for all the different options, rather then having lots of similar code.  The config setup is still too-fix

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

Updated the CollectHealth and CollectPerfMetrics config value types to boolean.
Added error check for condition where both the above values are disabled.
Added root privilege required info to plugin description and error messages.

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

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

check_uptime: New plugin, based on new cache_event callback.

dcpmm plugin collects performance and health statistics from Intel Optane DC Presistent Memory (DCPMM).

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

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

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