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

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.

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.

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>

resource_metrics.c: fix -Wpedantic warnings for casts, option 2

By changing compare function signatures to ones expected by
bsearch() & qsort().

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

resource_metrics.c: fix -Wpedantic warnings for casts, option 1

By casting function types to ones expected by bsearch() & qsort().

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

Pass strcmp() to c_avl_create() as function pointer, not data one

Matches how strcmp() is passed to c_avl_create() elsewhere.

Reported by -Wpendantic.

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

daemon/utils_cache.c: fix -Wpedantic warnings

Semicolons are nice for editor indenting, but redundant ones
are against ISO C.  This can be fixed by adding do/while(0) loop
to macro, which requires also making functions otherwise look
like normal functions.

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

Merge pull request #4190 from octo/6/swap

 [collectd 6] Swap plugin: Migrate to collectd 6 data structures.

swap plugin: Patch the "total" value through to `swap_append_usage3`.

src/collectd.conf.in: Fix default of `ReportBytes`.

collectd.conf(5): Improve wording for `ReportBytes`.

swap plugin: Call `plugin_dispatch_metric_family` unconditionally.

swap plugin: Rename `swap_submit_*` to `swap_append_*`.

swap plugin: Report utilization as fraction of one.

swap plugin: Populate the `unit` field.

swap plugin: Add the `ReportUsage` and `ReportUtilization` config options.

swap plugin: Add descriptions to metric families.

swap plugin: Use constants for label names and well-known label values.

swap plugin: Use different metric families for I/O reporting in bytes.

Depending on the "ReportIO" setting, the plugin may emit paging operations
or bytes. Use a different metric family for these two metrics.

This also moves the decision whether or not to report I/O to the
`swap_submit_io`. This fixes a bug on NetBSD where I/O was reported
even if "ReportIO" was set to false.

swap plugin: Group metrics into fewer metric families.

This should align this plugin with the OpenTelemetry semantic conventions:
https://opentelemetry.io/docs/specs/semconv/system/system-metrics/#pagingswap-metrics

Declare metric families as an array

Fix error message

ChangeLog: swap: plugin: migration to v6.0

write_redis plugin: Migrate to RedisTimeSeries.

write_redis plugin: Register metrics and resources only when necessary.

write_redis plugin: Add an intermediary set that maps resources to metrics.

write_redis plugin: Add unit test.

write_redis plugin: Use function pointers for reconnect, disconnect, and execute.

write_redis plugin: Update to work with `metric_family_t`.

format_json: Add the `format_json_metric_identity` function.

utils_cache: Add the `uc_get_first_time` function.

libplugin_mock: Link with the "real" cache.

Auto-Merge pull request #4252 from octo/6/df

Automatically merged due to "Automerge" label

fix: df plugin: Fix an array out of bounds crash.

The second argument of the `for` loop always evaluated to true, causing the
code to read past the end of the array.

ChangeLog: df plugin: An out of bounds read, causing segmentation faults, has been fixed.

Bump version to 6.0.0; Update ChangeLog.