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

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.

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.

Auto-Merge pull request #4251 from octo/6/cirrusci

Automatically merged due to "Automerge" label

ci: Remove the "statgrab" task from CirrusCI.

The "statgrab" job builds collectd with the "statgrab" library and then expects
certain plugins to be linked against it. However, since the build runs on
Linux, the plugins are actually using the "native" implementation. The linker,
noticing that the plugin is not referring to any symbol in the statgrab
library, does not actually produce a link, causing the test to fail.

The native implementations provide much richer metrics than the libstatgrab
based implementations, so we'll drop the test.

Merge pull request #4250 from octo/6/cirrusci

.cirrus.yml: statgrab: Print `ldd` of failing plugins. Continue on error.

.cirrus.yml: Verbose make build steps for the statgrab task.

.cirrus.yml: `$` is redundant for variable names within `$((...))`

... as shellcheck likes to often remind me.

.cirrus.yml: Use YAML multi-line string that preserves newlines.

.cirrus.yml: Run the non-standard tasks on PRs, too.

.cirrus.yml: statgrab: Print `ldd` of failing plugins. Continue on error.

Merge pull request #4198 from octo/6/import_plugins

[collectd 6] Import plugin changes from `main`.

intel_pmu plugin: Fix formatting of `counter_t` values.

contrib/format.sh src/cpufreq.c

src/vmem.c: Importing changes from `main`.

Changes:
9369ed71450430266ebd7ca410022c2b1b548fb4 vmem: add metrics starting with `pgrefill`
21ffe63374e08d9c3af28a0df33d46e4a908f84e vmem: add "scan" metrics starting with "pgscan_"
25364414f51603e6940fbc1853071e4bc0bb24aa vmem: add "pgscan_"
a8069c10a05d93b6aae0e52b1c6a14e54350fe6f vmem: add metrics start with "pgscan_" in Kernel Linux

src/virt.c: Importing changes from `main`.

Changes:
b4bdc7a8486683533185db34f97fd0f219fdc36b Fix the virt plugin for Openstack wallaby and later use (#4011)
8c781279e9c4789936323670b729f9a979c976db virt: Add hugetlb_ metrics

src/ubi.c: Importing changes from `main`.

Changes:
8c56291be070d7fcb99158157e1d42256dc4930e ubi: add percent evaluation for the bad blocks
d4e3332fc5198504483c0eac2b83b6cc48feb01a ubi: perpartion for percentage evaluation of the bad blocks

src/swap.c: Importing changes from `main`.

Changes:
284f51b21755693cb7cf927f002b26304fad7973 Removed extra call to swap_submit_usage.

src/smart_test.c: Importing changes from `main`.

Changes:
d228c3129c270754b61bf3caae0af36bc7a9c73a Fic smart_test.c for big endian. (#3799)

src/processes.c: Importing changes from `main`.

Changes:
d47b7020743e0ef942401b8cf143a0ac6476f9f5 clang format
71e99e00e21c8181408b590929f856830deac732 Compare both PID and process start time to avoid issues with immediate PID reuse on Linux platform.
9197134e7cd99206900f86f660be2135d028a2ab processes: format array size def in read fork_rate & ctxt
a8b4168ac2237449e3192be3dd2d07ef587fa3e2 processes: fix prints & remove useless comments
2495f75ea8fc4ac0f295b0bdd38cb5eeb1ddd7c4 processes: eliminate hardcoded arraysize in read ctxt & fork_rate
d2626382c6f0c506993ec13c4c2bcd69bad19371 processes: determine if ctxt is enabled before read_sys_ctxt_switch
0c41886ca975ed679e568210caa743af6bb75678 processes: format `/proc/stat` WARNING & ERROR
7a262979f76ebb5868320e09107bc02dc101ff07 processes: use buffer to read /proc/stat once according to procs_running
ffc486a93f980b4d0d859b5f627f859d1877646c processes: remove running++ since we calc it in procs_running later
c3bbd17e13558b15e09f0ecb374e44dfbf3623cd processes: make contextswitches configurable
dfd034032b7c7c8f821774715c0723c42cefd332 Add scalloc() and use it in src/processes.c (#4014)