write_prometheus plugin: Fix a deadlock due to a leaked mutex.

Merge pull request #4278 from octo/6/swap

[collectd 6] swap plugin: Add the `system.paging.faults` metric.

Merge pull request #4275 from octo/6/memory

[collectd 6] memory plugin: Report the `system.linux.memory.available` metric.

swap plugin: Add the `system.paging.faults` metric.

memory plugin: Report the `system.linux.memory.available` metric.

Update ChangeLog for version 6.0.0.rc2.

Add GitHub Action to automatically build and upload release assets.

Merge pull request #4274 from octo/6/cirrus_macos

.cirrus.yml: Add task for Mac OS X, cleanup and improvements.

version-gen.sh: Add the `--tags` flag to `git describe`.

.cirrus.yml: Use proper capitalization for MacOS and FreeBSD.

.cirrus.yml: Remove "or cat log" in favor of uploading logs as artifacts.

.cirrus.yml: Remove "clang" task.

It duplicates a GitHub Action based build.

cirrus.yml: Remove the "default toolchain" tasks.

The default toolchains are already covered well by GitHub Actions.

.cirrus.yml: Run `distcheck` with default configure flags.

.cirrus.yml: Build with `-gdwarf-4` when using clang.

.cirrus.yml: FreeBSD: install "git".

.cirrus.yml: Mac OS X: Pass `DEFAULT_CONFIG_OPTS` to configure.

.cirrus.yml: Disable `CIRRUS_CLONE_SUBMODULES` and run `git submodule` "manually".

.cirrus.yml: Use `${CIRRUS_CPU}` on Mac OS.

`nproc` is not available.

.cirrus.yml: Add macos_task for Mac OS X.

Merge pull request #4272 from octo/6/time_metrics

[collectd 6] Scale time metrics to seconds per second using floating point counters.

Merge pull request #4259 from octo/6/cirrusci

[collectd 6] ci: .cirrus.yml: Upload `.log` files as artifacts.

disk plugin: Report time metrics in seconds.

cpu plugin: Report `system.cpu.time` in seconds, using floating point counters.

common: Add support for floating point counters …

… to `rate_to_value()` and `value_to_rate()`.

Merge pull request #4266 from octo/6/fpcounter

[collectd 6] Introduce a floating point counter type.

Auto-Merge pull request #4270 from octo/6/format_stackdriver

Automatically merged due to "Automerge" label

format_stackdriver: Add proper error handling to `sd_output_reset()`.

format_stackdriver: Remove `int` cast from `yajl_gen_double()`.

Auto-Merge pull request #4268 from octo/6/format_stackdriver

Automatically merged due to "Automerge" label

format_stackdriver: When JSON generation fails, reset the generator because it is in an unknown state.

src/utils/value_list/value_list.h: Drop outdated comment.

format_open_telemetry: Move the "never reached" comment to `add_metric()`.

src/daemon/utils_cache.c: Avoid `default` in the `family->type` switch statement.

This way the compiler can check that we handle all values in the
`metric_type_t` enum.

src/daemon/metric.h: Add an `IS_CUMULATIVE` macro.

format_kairosdb: Log an error when `uc_get_rate()` fails.

format_json: Don't access `(value_t).gauge` when the type is `FPCOUNTER`.

src/daemon/metric.h: Order metric types by value.

value_list: Move `DS_TYPE_TO_STRING` out of `src/daemon/plugin.h`.

format_{graphite,influxdb,kairosdb,stackdriver}: Fix compiler warning.

Some compilers don't like empty statements, e.g.:

```
src/utils/format_kairosdb/format_kairosdb.c:122:8: error: label at end of compound statement
  122 |   case METRIC_TYPE_UNTYPED:
      |        ^~~~~~~~~~~~~~~~~~~
```

write_prometheus plugin: Add support for `METRIC_TYPE_FPCOUNTER`.

format_stackdriver: Add support for `METRIC_TYPE_FPCOUNTER`.

format_open_telemetry: Add support for `METRIC_TYPE_FPCOUNTER`.

format_kairosdb: Add support for `METRIC_TYPE_FPCOUNTER`.

format_json: Add support for `METRIC_TYPE_FPCOUNTER`.

format_influxdb: Add support for `METRIC_TYPE_FPCOUNTER`.

format_graphite: Add support for `METRIC_TYPE_FPCOUNTER`.

cmds: Add support for `METRIC_TYPE_FPCOUNTER`.

common: Add support for `METRIC_TYPE_FPCOUNTER`.

Daemon: Add a new metric type for floating point counters.

Merge pull request #4265 from octo/6/cache

[collectd 6] cache: Provide the first metric value available alongside the first metric time.

src/daemon/utils_cache_test.c: Add a test case for decreasing gauges.

format_stackdriver: Add missing `break` statements.

format_stackdriver: Consistently handle `METRIC_TYPE_UNTYPED` as an error.

src/daemon/utils_cache.h: Unify comment style.

format_stackdriver: Use the new `uc_first_metric()` function.

src/daemon/utils_cache.c: Let `uc_get_rate()` return early when called with a gauge metric.

src/daemon/utils_cache.c: Add documentation for `uc_get_rate()`.

This also improved the return values to give more information about which error condition occurred.

src/daemon/utils_cache.c: Change `uc_first_metric()` to return time and value.

Returning both in one call ensures these values are consistent, i.e. it avoids
the case that there might have been an update inbewtween reading time and
value.

src/daemon/utils_cache.c: Automatically initialize the cache when needed.

This simplifies writing unit tests using the cache.

chore: Add unit test for `uc_get_rate`.

Merge pull request #4264 from eero-t/gcc-extra

[collectd 6, WIP] Fix more GCC "-Wextra" warnings in collectd core

Merge pull request #4262 from eero-t/gcc-pedantic-func

[collectd 6, WIP] Fix more GCC "-Wpedantic" warnings

message_parser.c: fix -Wextra warnings

And arg name in comment for the function calling message parser
callback.

Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>

proc_pids.c: fix signed vs. unsigned comparison warnings

Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>

format_stackdriver.c: use counter_t instead of casts

All current uses are for counter_t comparisons, so let's just use that
type.  Gets rid of signed vs unsigned comparison warning.

Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>

mdevents.c: fix -Wpedantic warnings on array initialization

"{}" initializers are C23 feature, not C99 one:
https://en.cppreference.com/w/c/language/array_initialization

In first 2 cases initialization was redundant as "regerror()" and
"regexec()" fill the given arrays. Event initialization needed to be
fixed as last struct item writing is conditional.

Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>

rest of plugins: __FUNCTION__ -> __func__ (C99)

Fixes -Wpendantic warnings in rest of plugins.

Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>

dpdk*.c: __FUNCTION__ -> __func__ (C99)

Same -Wpendantic warnings fixes for plugin code.

Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>

utils/dpdk/dpdk.*: __FUNCTION__ -> __func__ (C99)

Fixes -Wpendantic warnings, __func__ supported since GCC 2.95.

Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>

utils/ovs/ovs.c: __FUNCTION__ -> __func__ (C99)

Fixes -Wpedantic warnings.

Collectd requires C99 support, and __func__ is supported since GCC 2.95 / C99.

Btw. For C++ code, the issue would be murkier. C++11 added __func__
support, but one cannot check just C++ version because MSVC reported
older spec until recently:
- https://devblogs.microsoft.com/cppblog/msvc-now-correctly-reports-__cplusplus/

More info on latter:
- https://stackoverflow.com/questions/7008485/func-or-function-or-manual-const-char-id
- https://learn.microsoft.com/en-us/cpp/cpp/func?view=msvc-170 +

Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>

utils/cmds/cmds_test.c: fix -Wpendantic/-Wformat warning

Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>

Merge pull request #4261 from eero-t/gcc-extra

[collectd 6]: Fix some GCC "-Wextra" warnings in collectd core

utils/metadata/meta_data.c: fix -Wnull-dereference warning

Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>

plugin.c: use union as generic callback type

Fix for -Wcast-function-type (part of -Wextra).

Fixes: d9d5025a0645
Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>

utils_cache.c: fix obsolete comment

Fixes: 95a54dd0f69c04
Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>

Merge pull request #4258 from octo/6/write_redis

[collectd 6] feat: Port the write_redis plugin to collectd 6.

.cirrus.yml: Rename `checks_script` to `make_distcheck_script`.

src/daemon/utils_cache.c: Add comment about initializing `ce->values_gauge` for non-gauge types.

Merge pull request #4257 from octo/6/dispatch_metric_family

[collectd 6] fix: Handle metric families with zero metrics gracefully.

src/daemon/plugin.c: Log a debug message when a metric family has no metrics.

ping plugin: Call `plugin_dispatch_metric_family` unconditionally.

irq plugin: Call `plugin_dispatch_metric_family` unconditionally.

ipc plugin: Call `plugin_dispatch_metric_family` unconditionally.

disk plugin: Call `plugin_dispatch_metric_family` unconditionally.

df plugin: Call `plugin_dispatch_metric_family` unconditionally.

chrony plugin: Call `plugin_dispatch_metric_family` unconditionally.

fix: Handle metric families with zero metrics gracefully.

This allows plugins to call `plugin_dispatch_metric_family()` unconditionally.

Update ChangeLog for version 6.0.0.rc1.

src/collectd.conf.pod: Fix typos and an incoherent sentence.

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

write_redis plugin: Avoid duplication of an int literal.

format_json: Use `STATIC_ARRAY_SIZE` when iterating over a static array.

write_redis plugin: Set the default timeout to one second.

This changes the behavior back to collectd 5 behavior.

.cirrus.yml: Upload `.log` files as artifacts.

Auto-Merge pull request #4243 from eero-t/gcc-pedantic

Automatically merged due to "Automerge" label

plugin.c: -Wpendantic cast fixes for callback handling

Remove void* _data_ pointer casts with correct function pointer types
and casts.  In some cases this allowed dropping the cast completely,
as I used same type for "cf_callback" member as the simplest init()
and shutdown() callbacks.

Only remaining pedantic warning is unfixable, as dlsym() API returns
void* data pointer, instead of function pointer.

Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>

vl_lookup_test.c: -Wpedantic lookup_create() arg cast fixes

Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>

value_list_test.c: add missing include

For STATIC_ARRAY_SIZE.

Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>

utf8_test.c: fix -Wpedantic sign issue

While GCC defaults to char being unsigned, tests would fail when
char is signed, so just declare strings to be uint8_t for utf8.

Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>

.gitignore: simplify + add more things to ignore

Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>