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)

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

Changes:
dcd22696ebeb4556ed172b0253b90d38bebfc742 use derive_t instead of int inside ovs_stats_get_port_stat_value
91a642385e2d068d106aa5c307de968f8b33add5 bug#3924 fix

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

Changes:
dd7e4fd34042483cc90ee2762f4b507fd6637d37 nginx: add support for unix domain sockets

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

Changes:
01412323b712d753cca9aada2e439f6c46e0d30d MySQL plugin: Fix incorrect WSREP/Galera metric types.
ca59e01d7fcf8cd802375b74053d87c24ed27017 add questions metric for mysql plugin

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

Changes:
0cac5b51626594c4e0291d03f06cc96eb5fe7d26 intel_pmu 2.0 - updates in collectd.conf.pod
549a22a8009ba82cf57aa70d1f3889a35a820042 intel_pmu 2.0 - change parameter name to AggregateUncorePMUs
2cd896a6c92629398fdffc559bdb5b578e107361 intel_pmu 2.0 - remove type from type instance
7e390a8231fddfe77085a5c68d2dee35fea78c54 intel_pmu 2.0 - bugfix & more verbose warning messages
9a483ae5b25d6da97d6bebdad2cd154a24274202 intel_pmu 2.0 - add name of pmu to plugin instance
16ed37e9b2b1a763b0236903ffa3087abbfc5cb5 intel_pmu 2.0 - Provide option to choose all events from EventList
e8e22b7a94f37ec5758f4f0bdcfc193dddc21130 intel_pmu 2.0 - Provide more data with metric
e838ecff613fd2ce255113cbfb4795c4b236d0ac intel_pmu 2.0 - updated version of plugin

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

Changes:
4cebbfc1ed4b44644d981df996a8ca941e38e8a1 Hugepages plugin skips reading write-only file

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

Changes:
5332f432633fb62fcd03b2540734b17e5785ab90 cpufreq: init max_avail_freqs with 128
d760819acbdc9a51c65b52c32cacec3ee9b991aa cpufreq: dynamically alloc max_avail_freqs according to time_in_state
45b676685b2a56545359c6b76985f95a56b99e72 cpufreq: format.sh cpufreq.c
31dbb8a334f9501662326496171f6cffd18bd92a cpufreq: make MAX_AVAIL_FREQS configurable in conf

src/collectd-nagios.c: Importing changes from `main`.

Changes:
d41fa79b7fda9b67601a7d7880d8e4b91a4500ce Feature/collectd nagios perfdata (#3879)

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

Changes:
93fbc158d09b85b98333f08ab16ce2c79f8271c4 Fix glibc feature macro handling for timegm

Merge pull request #4246 from octo/6/memory

[collectd 6] docs: Fix the default value of the memory plugin's `ReportLimit` option.

docs: Fix the default value of the memory plugin's `ReportLimit` option.

Merge pull request #4217 from octo/6/disk

[collectd 6] disk plugin: Align metrics with OpenTelemetry recommendations.

disk plugin: Do submit metrics in the first iteration.

disk plugin: Remove the "avg time per op" calculation.

The result was unused.

disk plugin: Revamp the Linux code.

*   Use `counter_diff` to calculate counter differences.
*   Use `counter_t` as we actually want the counter overflow behavior for these metrics.
*   Remove the `has_...` fields from the global data struct.
*   Use `value_to_rate()` to calculate disk busyness.
*   Use `strtoull()` to parse counter values.

disk plugin: Add `system.disk.io_time`, `system.disk.weighted_io_time`, and `system.disk.pending_operations` for Solaris.

disk plugin: Consistently scale the `fam_disk_io_time` metric to microseconds.

disk plugin: Improve metric descriptions.

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

disk plugin: Remove ops_time metric for Solaris.

The fields do not mean what we thought they meant. "wtime" means "wait time",
"rtime" means "run time".

Fixes: #3875 (for collectd 6)

disk plugin: Scale all time counters to microseconds.

Also detone the time scale in the variable names.

disk plugin: Use constants for the "direction" label values.

disk plugin: Fix incorrect type.

disk plugin: Align metrics with OpenTelemetry recommendations.

* Rename labels to `system.device` and `disk.io.direction`.
* Rename `system.disk.time` to `system.disk.operation_time`.
* Add descriptions and units to all metric families.
* Add the "utilization" metric to FreeBSD.

Merge pull request #4216 from octo/6/cpu

[collectd 6] cpu plugin: Align metrics with OpenTelemetry recommendations.

cpu plugin: Fix function comment.

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

cpu plugin: Improve error checking; define and use `USAGE_UNAVAILABLE`.

cpu plugin: Improve the description of the `system.cpu.logical.count` metric.

cpu plugi: Consistently set the `finalized` field at the end of the function.

cpu plugin: Sort CPU states alphabetically.

cpu plugin: Add "ReportUsage" to `config_keys`.

Also sort `config_keys`.