write_riemann plugin: Terminate `riemann_event_set` arguments with `RIEMANN_EVENT_FIELD_NONE`.

`riemann_event_set` is a variadic function, that means it accepts a variable
number of arguments. That means it needs some way to determine – at runtime
– how many arguments there are. It appears to be doing so by using
`RIEMANN_EVENT_FIELD_NONE` to indicate the last element in the argument
list. Unfortunately I was unable to find the library's documentation and
code and could not verify this.

That means that the argument list passed to `riemann_event_set` was not
always terminated, causing it to read past where it was supposed to and
adding random crap into the message it crafted.

Issue: #4050

Merge pull request #3779 from clear-code/main-more-lua-callbacks

lua: Add functions to register some missing callbacks

lua: Add functions to register some missing callbacks

* register_init
* register_shutdown
* register_config

In this version, the configuration for each Script is
supported.

  Script "foo.lua"
  Script "bar.lua"
  <Module "foo.lua">
    Key1 Value1
  </Module>
  <Module "bar.lua">
    Key2 Value2
  </Module>

The scope of accessible key-value pair is limited to under <Module>
This feature is similar to collectd-python.

  ref. https://www.collectd.org/documentation/manpages/collectd-python.html

Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>

Merge pull request #3777 from clear-code/change-lock-scope

lua: Use a global lock to call Lua's API

Merge pull request #3778 from clear-code/detect-luajit

lua: support to detect LuaJIT automatically

lua: Use a global lock to call Lua's API

Since vanilla Lua isn't multi-thread-safe, a global lock should be used
when calling their functions. The previous code causes unstable behavior
especially on loading Lua's C modules with both read & write threads.

e.g.)

  collectd.register_read(function()
    require('cqueues')
    return 0
  end)

  collectd.register_write(function()
    return 0
  end)

Note that you need to add the following config to your collectd.conf to
load C modules:

  <LoadPlugin lua>
    Globals true
  </LoadPlugin>

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

Merge pull request #4022 from matwey/epics

EPICS plugin

Initial implementation for epics plugin

EPICS is Experimental Physics and Industrial Control System.

Reference: https://epics-controls.org/

Signed-off-by: Matwey V. Kornilov <matwey.kornilov@gmail.com>

lua: add fallback to detect LuaJIT

Before:
  LIBLUA_PKG_CONFIG_NAME=luajit ./configure --enable-lua

After:
  ./configure --enable-lua

In the previous versions, for enabling LuaJIT,
it needs to specify LIBLUA_PKG_CONFIG_NAME=luajit explicitly.

In this commit, add fallback to detect LuaJIT without specifying
LIBLUA_PKG_CONFIG_NAME=luajit explicitly by default (just use
--enable-lua).

Note that if both of lua and luajit exists, then lua is used.

Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>

collection3: Fix inappropriate usage of CGI:param

When using collection3 as a CGI, the following error is sent to logs
repeatedly.

  FastCGI sent in stderr: "CGI::param called in list context from
  /usr/share/doc/collectd-core/examples/collection3/lib/Collectd/Graph/Common.pm
  line 529, this can lead to vulnerabilities. See the warning in
  "Fetching the value or values of a single named parameter" at
  /usr/share/perl5/CGI.pm line 412"

This is caused inappropriate usage of param(), it should be handled as
a scalar or should be treated by multi_param() explicitly.

Signed-off-by: Kentaro Hayashi <kenhys@gmail.com>

Merge pull request #4205 from octo/putval

cmds: Add a unittest for `cmd_parse_putval`.

common: Use `sstrncpy` to get around misinformed compiler error.

common: Fix compiler warning in unit test.

Some compilers complained about using a value that depends on the input for the
size argument passed to `strncpy`. They missed that the size of `buf` also
depends on the source argument and therefore this use is fine.

```
  CC       src/utils/common/common_test.o
../../src/utils/common/common_test.c: In function 'test_parse_values':
../../src/utils/common/common_test.c:316:5: error: 'strncpy' specified bound depends on the length of the source argument [-Werror=stringop-overflow=]
  316 |     strncpy(buf, cases[i].buffer, buf_sz);
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../../src/utils/common/common_test.c:314:21: note: length computed here
  314 |     size_t buf_sz = strlen(cases[i].buffer) + 1;
      |                     ^~~~~~~~~~~~~~~~~~~~~~~
```

contrib/format.sh src/utils/common/common_test.c

cmds: Add a test for `cmd_parse_putval`.

cmds: Prevent `cmd_parse_putval` from modifying its `argv` argument.

common: Change `parse_values` so it doesn't modify the parsed string.

Merge pull request #4184 from octo/issue/4137

MySQL plugin: Fix incorrect WSREP/Galera metric types.

MySQL plugin: Fix incorrect WSREP/Galera metric types.

Fixes: #4137

Merge pull request #4065 from carlospeon/flush-buffer

evaluate flush condition with oldest value in the buffer

evaluate flush condition with oldest value in the buffer

Merge pull request #4178 from octo/convigure-virt

configure: Improve the reporting when the "virt" plugin cannot be built.

Makefile.am: Build `test_virt_plugin` with the same `CFLAGS` as the *virt plugin*.

Using different flags leads to build errors if libvirt is in a non-system location.

configure: Use `elif` instead of two separate `if` blocks.

configure: Improve the reporting when the "virt" plugin cannot be built.

Merge pull request #4172 from octo/procevent

procevent plugin: Remove embedded flexible array member to fix compiler warning.

procevent plugin: Consistently mention the unit in log message.

Co-authored-by: Eero Tamminen <eero.t.tamminen@intel.com>

procevent plugin: Reference buffer sizing guideline.

Co-authored-by: Eero Tamminen <eero.t.tamminen@intel.com>

write_log plugin: remove the "write_log values:" line when logging metrics.

procevent plugin: report errors in human readable form.

procevent plugin: ensure the ring buffer length is not zero.

The manpage documents a default of 10, which was not implemented. This
change brings the implementation in line with the documentation.

procevent plugin: remove use of a nested flexible array member.

The previous code used an ad-hoc struct to construct or parse a Netlink
message. This relied on allocating a field _after_ the struct with the
flexible array member, which is prohibited by the C standard, leading to
compiler warnings.

Merge pull request #4171 from octo/workflow

Improve CI workflows

Merge pull request #4164 from octo/notify_upstart_systemd

Notify upstart/systemd after initialization is complete.

Build workflow: request that clang produce DWARF v4 debug symbols.

Disable "exit on error" when running `make check`.

procevent plugin: Disable the `-Wgnu-variable-sized-type-not-at-end` warning.

.cirrus.yml: Update list of compilers to match the `debian_unstable` container.

Build workflow: add builds for `--enable-debug` and `CC=clang`.

Merge pull request #4170 from octo/debian_sid

Workflows: rename "sid_amd64" to "debian_unstable".

Workflows: rename "sid_amd64" to "debian_sid".

Merge pull request #4166 from octo/cirrus

Improve tests run on the Cirrus CI platform.

.cirrus.yml: Report failures from CentOS 9 and Fedora 39.

.cirrus.yml: Fix reporting of test failure.

.cirrus.yml: set parallelism to number of CPUs.

.cirrus.yml: Add task for FreeBSD builds.

.cirrus.yml: remove obsolete platforms, add recent ones.

.cirrus.yml: s/master/main/

Merge pull request #4169 from octo/nut/upsclient

NUT plugin: improve support for faulty versions of libupsclient.

NUT plugin: import `<time.h>` before `<upsclient.h>`.

Merge pull request #4167 from octo/ganglia

gmond plugin: Import `<ganglia.h>`.

gmond plugin: Improve `mc_handle_metadata_msg()`.

This function was not properly formatted. While there:

* Handle the error instead of indenting the successful flow.
* Declare variables late.
* Initialize variables when declaring them.

gmond plugin: Look for RPC headers in /usr/include/tirpc.

glibc removed the SunRPC headers, and the replacement headers are in a
different location. Try and look for them there.

gmond plugin: Import `<ganglia.h>`.

`<gm_protocol.h>` has been removed in new versions of libganglia.

Merge pull request #4168 from octo/workflow/upload

Workflow build: upload `config.log` and other log files.

Workflow build: upload `config.log` and other log files.

Notify upstart/systemd after initialization is complete.

Fixes: #4152

Merge pull request #4165 from octo/netlink/valgrind_issue

Netlink plugin: complete initialize structs used for testing.

SMART plugin: initialize struct passed to `ioctl(2)`.

Valgrind is complaining about a conditional jump based on uninitialized
memory:

```
==66462== Conditional jump or move depends on uninitialised value(s)
==66462==    at 0x10C500: smart_read_nvme_intel_disk (in /__w/collectd/collectd/test_plugin_smart)
==66462==    by 0x10D366: test_x (in /__w/collectd/collectd/test_plugin_smart)
==66462==    by 0x10D638: main (in /__w/collectd/collectd/test_plugin_smart)
```

This may be due to the `struct nvme_additional_smart_log` being
uninitialized when it's being passed to `ioctl(2)`.

This there, this removed an unnecessary level of indentation.

Netlink plugin: complete initialize structs used for testing.

Valgrind complains about a conditional jump based on uninitialized
memory:

```
==66438== Conditional jump or move depends on uninitialised value(s)
==66438==    at 0x10CA06: vf_info_submit (in /__w/collectd/collectd/test_plugin_netlink)
==66438==    by 0x1110F2: test_vf_submit_test (in /__w/collectd/collectd/test_plugin_netlink)
==66438==    by 0x112EAC: main (in /__w/collectd/collectd/test_plugin_netlink)
```

This is likely caused by the `vf_stats_t` being only partially
initialized. Using a struct initializer is not only cleaner, it also
ensures the remainder of the struct is initialized to zero.

Merge pull request #4163 from octo/workflows/fedora39

Build workflow: Add Fedora 38 and 39.

Build workflow: report the status of `make check` after the test log has been uploaded.

Build workflow: use all available CPUs.

Build workflow: Add Fedora 38 and 39, remove Fedora 36.

Merge pull request #4162 from octo/workflows/build

Build workflow: remove old distros.

./contrib/format.sh src/amqp.c

Update README.md

Add Ubuntu 22.04 ("Jammy Jellyfish") and Ubuntu 23.10 ("Mantic Minotaur").

Build workflow: also remove EL8 (based on CentOS 8).

Build workflow: remove old distros.

This removes old and unsupported Linux distros:

*   Ubuntu 16 ("Xenial", support ended April 2021)
*   Ubuntu 18 ("Bionic Beaver", support ended June 2023)
*   Enterprise Linux 7 (technically still supported, but nine years old
    at this point)
*   Fedora 34 (support ended June 2022)
*   Fedora 35 (support ended December 2022)

Fedora 36 is also unsupported. We will keep the builder until we have
never versions of Fedora available (or at least Rawhide builds again).

Merge pull request #4111 from mrunge/rabbitmq

Fix compile warning for amqp

AMQP plugin: prefer the `rabbitmq-c/amqp.h` header if available.

At some point, RabbitMQ has moved their headers to the `rabbitmq-c/`
subdirectory. The old locations still exist but throw an error, saying
the old headers are "deprecated".

This adds appropriate checks for the new headers to the configure script
and uses those if present.

To simplify both the configure script and plugin, support for ancient
versions of the library is removed. This affects versions that don't
have the `amqp_tcp_socket_new()` function yet.

Fix compile warning for amqp

warning "amqp.h is deprecated, use rabbitmq-c/amqp.h instead."

Merge pull request #4161 from octo/intel_rdt/pids2

Intel RDT plugin: use `pqos_mon_start_pids2()` if available.

Intel RDT plugin: use `pqos_mon_start_cores()` if available.

Intel RDT plugin: use `pqos_mon_start_pids2()` if available.

Newer versions of the pqos library deprecate the previous function, so
we have to migrate even though it makes memory management quite ugly.

ChangeLog: Intel RDT plugin: support for PQOS ≥5.0.0 has been added.

Merge pull request #4160 from octo/python/SetArgv

Python plugin: use `PyConfig` to set argv.

Python plugin: use `PyConfig` to set argv.

`PySys_SetArgv` has been deprecated in Python 3.11.

Merge pull request #4158 from octo/write_mongodb/format-truncation

Write MongoDB plugin: fix format trunction errors.

Write MongoDB plugin: fix format trunction errors.

When a `size_t` is a 64bit integer, the 16 byte buffer could potentially
overflow. Fix this by using a larger buffer and also collectd's own
`ssnprintf()` function which guarantees to result in a null terminated
string.

```
In function 'wm_create_bson',
    inlined from 'wm_write' at src/write_mongodb.c:229:17:
src/write_mongodb.c:99:32: error: '%zu' directive output may be truncated writing between 1 and 20 bytes into a region of size 16 [-Werror=format-truncation=]
   99 |     snprintf(key, sizeof(key), "%" PRIsz, i);
      |                                ^~~
src/write_mongodb.c: In function 'wm_write':
src/write_mongodb.c:99:33: note: format string is defined here
   99 |     snprintf(key, sizeof(key), "%" PRIsz, i);
```

Merge pull request #4157 from octo/distcheck

Add new GitHub action for running `make distcheck`.

Give the action and job better names.

Remove `make distcheck` from the regular "build" action.

Distcheck action: use Debian 11 ("Bullseye") for now.

Debian 12 does not compile yet.

Add new GitHub action for running `make distcheck`.

Merge pull request #4156 from octo/curl_off_t

curl_stats: fix compatibility with new versions of cURL.

curl_stats: fix compatibility with new versions of cURL.

Use integer based keys for metrics if available.

cURL ≥ 7.55.0 provides additional keys that allow getting certain
metrics as integers rather than doubles, e.g. content length. In some
newer versions of cURL, the original keys (using doubles) are marked as
deprecated.

ChangeLog: cURL, cURL-JSON, cURL-XML, Write HTTP plugins: fix compatibility with new versions of cURL.

Build workflow: add Debian 12 ("Bookworm") and remove Debian 9 ("Stretch").

Merge pull request #4155 from octo/valgrind

Add Valgrind suppression for libvirt memory leak.

Add Valgrind suppression for libvirt memory leak.

Merge pull request #4154 from octo/intel_rdt

Intel RDT: fix compilation with libpqos < 4.4.

Intel RDT: fix compilation with libpqos < 4.4.

Revert "GitHub actions: upgrade to actions/checkout@v4"

This reverts commit 8a451bca30011fbfd249bbd3aa78922cb03aef2d.

Some older platforms use an incompatible glibc version.

GitHub actions: upgrade to actions/checkout@v4

GitHub actions: include job name in artifact name.

Otherwise a random job would "win" and the entire workflow only produces one artifact.

GitHub actions: continue on error and upload test logs.

When `make check` fails, continue so that the "upload logs" step is
performed. Use the `upload-artifact` action instead of just `cat`ing the
file.

Merge pull request #3773 from drNowak/bugfix-negative-cpu

Compare both PID and process start time to avoid issues with immediate PID reuse on Linux

clang format

Compare both PID and process start time to avoid issues with immediate PID reuse on Linux platform.

write_prometheus: don't use AI_ADDRCONFIG for resolving bind address

Fixes #4150

write_prometheus uses getaddrinfo to resolve the bind address.
The AI_ADDRCONFIG flag causes getaddrinfo to refuse to resolve
0.0.0.0 when the system has no non-loopback IPv4 addresses configured
and refuse to resolve :: when the system has no non-loopback IPv6 configured.

We want binding to a wildcard address (0.0.0.0 or ::) to always work,
even if the network is down.

To achieve that, don't pass the AI_ADDRCONFIG flag
when resolving a bind address.