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

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`.

cpu plugin: Apply code review suggestions.

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

cpu plugin: Use a const instead of repeating a literal multiple times.

cpu plugin: Fold `usage_global_rate` into `usage_rate`.

cpu plugin: Remove the special "global" case from `usage_ratio`.

This will just work transparently.

cpu plugin: Fold the `usage_global_count` function into `usage_count`.

cpu plugin: Rename `has_value` to `has_rate`.

This way the use of the field is much easier to understand when reading the
code.

cpu plugin: Remove tests for `usage_global_ratio` and `usage_global_count`.

The functionality is tested in the test cases for `usage_ratio` and
`usage_count` and there is no need to test these separately. The opposite: the
rest of the CPU plugin only uses `usage_ratio` and `usage_count`, so testing
the global variants leaks abstraction.

cpu plugin: Fold testing for `CPU_ALL` into the tests for `usage_ratio` and `usage_count`.

cpu plugin: Remove unused variable.

cpu plugin: Link the unit test with libstatgrab, if so configured.

cpu plugin: Skip states that don't have any data.

cpu plugin: Use the `cpu_num` field in `usage_t`.

cpu plugin: Add a `cpu_num` field to `usage_t`.

cpu plugin: Update code to use the `usage_t` type.

cpu plugin: Move type definitions close together.

cpu plugin: Implement `usage_global_count()`.

cpu plugin: Implement `usage_count()`.

This one is done in a second aggregation loop, because we require a CPU-level
aggregate rate to be available to properly scale the counter.

cpu plugin: Move aggregation into a central finalize() function.

cpu plugin: Implement `usage_global_ratio()`.

cpu plugin: Test that `usage_rate(STATE_ACTIVE)` equals `usage_active_rate()`.

cpu plugin: Implement `usage_ratio`.

cpu plugin: Add `usage_global_rate`.

cpu plugin: Aggregate all non-idle states into the "active" state.

cpu plugin: Ensure unpopulated states return NAN.

cpu plugin: Add a very simple `usage_t` type for aggregation.

cpu plugin: Simplify the configuration options available.

 *  The options `ReportUsage`, `ReportUtilization`, and `ReportNumCpu` control
    which metrics are emitted on a high level. Other options no longer influence
    *what* is being collected. This also allows to report usage and utilization
    metrics simultaneously.
 *  The documentation has been updated to reflect that the plugin no longer
    emits a percentage, but a ratio for utilization metrics.

cpu plugin: Remove overly verbose prefix from "state" constants.

cpu plugin: Use constants for label names.

cpu plugin: Align metrics with OpenTelemetry recommendations.

* Add metric description and unit.
* Update label names (e.g. "cpu" → "system.cpu.logical_number")
* Divide rates by number of CPUs, so that the sum of all rates equals to 1.
  (Previously the sum of all rates was equal to the number of logical CPUs)
* Remove the "cpu=total" label when aggregating CPUs.

Merge pull request #4239 from eero-t/pkg-config-l0

gpu_sysman: enable plugin automatically when `pkg-config` finds `level-zero` dependency

gpu_sysman: change configure to use pkg-config for dependency

And enable Sysman plugin automatically when dependency is found.

While GPU plugin uses just Sysman API subset of the Level-Zero API
family, name of the used library / loader is level-zero => dropped
"--with-sysman" option and renamed configure variables.

Also, now that Level-Zero packages are in the distributions, Sysman
API GPU plugin could be enabled by default when pkg-config finds
level-zero. This simplifies config code.

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

Merge pull request #4244 from eero-t/its-history

[collectd 6] remove unused uc_get_history() from src/daemon/utils_cache.c

src/daemon/utils_cache.c: remove unused uc_get_history()

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

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

Automatically merged due to "Automerge" label

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