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.

df plugin: Use constants for state label values.

df plugin: Report filesystem mode (ro/rw).

df plugin: Use `STATIC_ARRAY_SIZE` to iterate over metric families.

df plugin: Update metric label names to match OpenTelemetry.

df plugin: Populate the `unit` field.

This also changes the report of usage to a ratio, as recommended by
OpenTelemetry. Previously we reported a percentage.

df plugin: Correctly report inode utilization.

The metrics were attached to the wrong metric family previously.

Merge pull request #4236 from octo/6/compat

[collectd 6] feat: Add the `--enable-compatibility-mode` flag to the `configure` script.

CI: Pass the `--enable-compatibility-mode` to all builds.

configure.ac: Disable plugins using `value_list_t` by default.

Merge pull request #4224 from octo/6/memory

[collectd 6] Memory plugin: Add "shared", drop "slab" and "available", report utilization as ratio.

memory plugin: Update return value consistently.

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

memory plugin: Sort list of states alphabetically.

memory plugin: Add comment about the other 2 options being legacy ones.

memory plugin: Report "shared" memory (Linux only).

memory plugin: Remove "slab" and "available" memory.

Under Linux, the plugin reported "slab" (kernel data structures cache) and
"available" (estimate of how much memory might be available without swapping).
These metrics group memory along different dimensions than other reported
memory, leading to double counting of some memory. The result was that the sum
of all memory metrics was not constant.

memory plugin: Remove overly verbose prefix from enum values.

memory plugin: Add the "system.memory.limit" metric.

memory plugin: Enable utilization reporting by default.

This follows OpenTelemetry semantic convention.

memory plugin: Rename `ValuesPercentage` to `ReportUtilization`.

memory plugin: Rename `ValuesAbsolute` to `ReportUsage`.

memory plugin: Report a fraction of 1 rather than a percentage.

write_riemann plugin: Terminate `riemann_event_set` arguments with `RIEMANN_EVENT_FIELD_NONE`.

`riemann_event_set` is a variadic function, that means it accepts a variable
number of arguments. That means it needs some way to determine – at runtime
– how many arguments there are. It appears to be doing so by using
`RIEMANN_EVENT_FIELD_NONE` to indicate the last element in the argument
list. Unfortunately I was unable to find the library's documentation and
code and could not verify this.

That means that the argument list passed to `riemann_event_set` was not
always terminated, causing it to read past where it was supposed to and
adding random crap into the message it crafted.

Issue: #4050

Merge pull request #4228 from octo/6/interface

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

interface plugin: Add link to spec.

interface plugin: Fix variable name: `fam` -> `fams`.

interface plugin: Align metrics with OpenTelemetry recommendations.

ChangeLog: Interface plugin: The metric schema has been aligned with OpenTelemetry semantic conventions.

Merge pull request #4233 from octo/6/strncpy

[collectd 6] feat: Improve `sstrncpy` semantic.

common: Overhaul the `sstrncpy` implementation.

Properly check all arguments and behave in a sane manner, i.e. don't crash.

I went back and forth a few times on whether to return `NULL` or `dest` when `n == 0`.

*   On the one hand, `n == 0` is not really an error and a situation that could
    naturally occur, e.g. when you're implementing code that appends to the end
    of a string.
*   On the other hand, if we return `NULL` when `n` is zero we can guarantee
    that we will either return `NULL` or a null terminated string.

Ultimately I decided to go with the stronger guarantee, i.e.

```c
if (n == 0) {
  return NULL;
}
```

common: Reserve a null byte when calling `strncpy`.

While `sstrncpy` guarantees a null terminated string, some compilers don't get
the memo and complain about the buffer size being equal to the size provided to
*strncpy(3)*. This *is* a potential source of error with *strncpy(3)*, because
if the source string is longer than the buffer, the buffer is not null
terminated. That is the precise reason `sstrncpy` exists in the first place.

Make these compilers happy by decreasing the size passed to *strncpy(3)* by
one.

Merge pull request #3795 from kenhys/v6-lua

[collectd 6] lua: migrate the Lua plugin to v6.0

lua: update data types section

It describes changes about internal data type.

Signed-off-by: Kentaro Hayashi <kenhys@gmail.com>

lua: migrate interface to v6 plugin API

Since v6, value_list_t is deprecated and metric_family_t is
introduced.

metric_family_t is mapped to the following Lua table:

  {
    name => ...,
    help => ...,
    unit => ...,
    type => ...,
    resource => {
      key1 => value1,
      ...,
      keyN => valueN
    }
    metric => {
     [1] => {
       label => {
         key1 => value1,
         ...,
         keyN => valueN
       }
       value => ...,
       time => ...,
       interval => ...,
       meta => {
         key1 => value1,
         ...,
         keyN => valueN
       }
     },
     ...
     [N] => {
       ...
     }
  }

Closes: #3654
Signed-off-by: Kentaro Hayashi <kenhys@gmail.com>

lua: enable Lua

Signed-off-by: Kentaro Hayashi <kenhys@gmail.com>

Auto-Merge pull request #4231 from octo/6/submodules

Automatically merged due to "Automerge" label

.gitmodules: Defaut to shallow submodules.

.github/workflows/: Only activate the opentelemetry submodule.

memory plugin: Use a constant for label names.

Merge pull request #4180 from octo/open-telemetry

[collectd 6] Write OpenTelemetry: New plugin to export metrics via OTLP over gRPC.

write_open_telemetry plugin: Fix minor memory leak.

Error:

```
==171106== 8 bytes in 1 blocks are definitely lost in loss record 58 of 228
==171106==    at 0x48407B4: malloc (vg_replace_malloc.c:381)
==171106==    by 0x492E7F9: strdup (strdup.c:42)
==171106==    by 0x1131F9: cf_util_get_string (configfile.c:1127)
==171106==    by 0x4EA3FD0: ot_config_node (write_open_telemetry.cc:206)
==171106==    by 0x4EA3FD0: ??? (write_open_telemetry.cc:262)
==171106==    by 0x111794: dispatch_block_plugin (configfile.c:464)
==171106==    by 0x1119B9: dispatch_block (configfile.c:508)
==171106==    by 0x11313D: cf_read (configfile.c:1100)
==171106==    by 0x11034E: configure_collectd (collectd.c:356)
==171106==    by 0x1104B6: init_config (collectd.c:406)
==171106==    by 0x121F3F: main (cmd.c:167)
```

src/collectd.conf.in: Add a `LoadPlugin` line for write_open_telemetry.

format_open_telemetry: Add `return` statement to that the block is not empty.

Write Open Telemetry plugin: Improve readability of time comparison.

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

format_open_telemetry: Handle unexpected metric types gracefully.

format_open_telemetry: Unify the `set_gauge` and `set_sum` functions.

format_open_telemetry: Reuse local variable.

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

src/daemon/metric.c: Drop redundant check from `metric_family_clone`.

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

format_open_telemetry: Populate the `unit` field.