.expected-plugins/debian_unstable: Import expected plugins list.

.expected-plugins/fedora_main: Import expected plugins list.

.github/workflows/build.yml: Add "Check expected plugins" to "experimental".

.expected-plugins/fedora38: Import expected plugins list.

.expected-plugins/ubuntu20.04: Import expected plugins list.

.expected-plugins/ubuntu22.04: Import expected plugins list.

.expected-plugins/debian10: Import expected plugins list.

.expected-plugins/debian11: Import expected plugins list.

.github/workflows/build.yml: Add a "Check expected plugins" step.

.expected-plugins/centos9: Import expected plugins list.

.expected-plugins/fedora39: Import expected plugin list.

.expected-plugins/ubuntu23.10: Import expected plugins list.

.expected-plugins/debian12: Import expected plugins list.

.expected-plugins: Add empty files for all distros.

.cirrus.yml: Provide expected plugin list to `check-built-plugins.sh` if present.

.cirrus.yml: Rename container images.

* bookworm_amd64 → debian12
* mantic_amd64 → ubuntu23.10
* el9_x86_64 → centos9
* fedora_rawhide_x86_64 → fedora_main

.github/workflows/distcheck.yml: Run on Debian12 (current stable).

.github/workflows/: Rename container images.

* bookworm_amd64 → debian12
* bullseye_amd64 → debian11
* buster_amd64 → debian10
* mantic_amd64 → ubuntu23.10
* jammy_amd64 → ubuntu22.04
* focal_amd64 → ubuntu20.04
* el9_x86_64 → centos9
* fedora38_x86_64 -> fedora38
* fedora_rawhide_x86_64 → fedora_main

Merge pull request #4210 from eero-t/min-mem-fix

[collectd 6] gpu_sysman: fix minimum memory usage calculation

gpu_sysman: fix minimum memory usage calculation

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

Merge pull request #4206 from octo/6/write_prometheus

[collectd 6] Write Prometheus plugin: Ensure metric and label names are properly formatted.

contrib/format.sh src/utils/strbuf/strbuf.c

src/utils/strbuf/strbuf.c: Remove unused left-over code.

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

strbuf: Rewrite `strbuf_print_restricted` using a bitmap.

Instead of building the buffer piece by piece, copy the entire string into
the buffer and do the replacements there.

I think the code is quite efficient, but I haven't profiled either
version so can't say for sure that there is a speedup. The new code
*may* be easier to reason about, since the "copy and replace" approach
has a much simpler loop body than the previous approach.

./contrib/format.sh src/utils/strbuf/strbuf.c

write_prometheus plugin: Replace invalid characters in names.

write_prometheus plugin: Add two more simple unit test cases.

write_prometheus plugin: Add an initial (i.e. very simple) unit test.

write_prometheus plugin: Break out formatting of a single metric family.

configure: Report why the *write_prometheus plugin* is disabled.

strbuf: Add `strbuf_print_restricted`.

This new function is similar to `strbuf_print_escaped` but differs in two
important aspects:

*    `strbuf_print_restricted` expects a list of acceptable characters,
    i.e. an allow list. `strbuf_print_escaped` expects a deny list.
*    `strbuf_print_restricted` *replaces* characters not in the allow list.
     `strbuf_print_escaped` adds an escape character in front of the
    denied character.

Merge pull request #4207 from octo/6/makefile_am

[collectd 6] Link more things into `libplugin_mock.la`.

Makefile.am: Add `src/daemon/{configfile,types_list}.c to `libplugin_mock`

… and remove from tests referencing these files directly.

Makefile.am: Link `libmetric` and `liboconfig` into `libplugin_mock`.

This allows us to remove these as direct dependencies from many tests.

Because `libmetadata` is linked into `libmetric`, it can also be removed.

Overall this will make link more things that are in the daemon into
`libplugin_mock`, making linking unit tests simpler.

Merge pull request #4201 from octo/6/ci

[collectd 6] .github/workflows/build.yml: Remove Ubuntu Bionic.

.github/workflows/build.yml: Remove Ubuntu Bionic.

It is too old to work with `actions/checkout@v4`.

Merge pull request #4203 from octo/6/cpu_sysman

[collectd 6] gpu_sysman plugin: Fix double reporting of metric.

gpu_sysman plugin: Fix double reporting of metric.

This metric is already added to the metric family in line 1553. Adding
it twice leads to `uc_update` errors.

Fixes: #4200

Merge pull request #4197 from octo/6/legacy_support

[collectd 6] Generate better names and labels for legacy metrics.

value_list: Simplify the control flow when setting metric labels.

value_list: Simplify by adding separating "." in the first `strbuf_print()` call.

Co-authored-by: Florian Forster <octo@collectd.org>

./contrib/format.sh src/utils/common/*.[ch]

value_list: Add special handling for "utilization" and "io" metrics.

common: Add `string_has_suffix`.

value_list: Ensure that the plugin name is contained in the converted metric.

value_list: Improve generated metric names.

This changes `plugin_value_list_to_metric_family` to generate metric
names that are more in line with OpenTelemetry's naming guidelines.

Merge pull request #4195 from octo/6/value_list

[collectd 6] Move legacy code into its own package.

./contrib/format.sh src/utils/value_list/value_list_test.c

curl_json plugin: Remove `value_list` from build deps.

Link `value_list` into `libplugin_mock`.

This is much simpler than adding the .c and .h files into all individual
tests.

intel_rdt plugin: Add the `value_list` utility to the unit test.

value_list: Migrate `parse_values` out of `common`.

value_list: Migrate functions related to `value_list_t` out of `common`.

daemon: Move `data_set_t` and related functions to its own file.

This is part of the legacy support for collectd 5 metrics. Because "data
sets" are a global resource, this has to remain within the daemon.

Move functions related to `value_list_t` to a separate package.

Merge pull request #4196 from octo/6/resource_metrics

[collectd 6] New utility: `resource_metrics` to group metrics by resource.

resource_metrics: Skip duplicate metrics instead of erroring out.

The semantics have been changed to simply ignore metrics that are already
in the set. The previous semantic was optimized for a "add and if that
fails flush and try again" plugin behavior. With the support for multiple
instances of the same metric (at different times), there no longer is a
need to ensure metrics in the set are unique.

resource_metrics: Treat metrics with different time stamps as different metrics.

resource metrics: Add missing free to `resource_metrics_reset`.

resource_metrics: Doc comment improvements.

resource_metrics: Fix bugs surfaced by the unit test.

* Sort metric families after inserting.
* Dereference the pointer-pointer returned by `bsearch` in `lookup_family`.
* Delete the metrics created by `metric_family_clone`.
* Run ./contrib/format.sh

resource_metrics: Add unit test.

New utility: resource_metrics.

Resource metrics allows plugin to stage metric families, grouped by
resource. This is designed to work well with plugins that export the
OpenTelemetry protocol.

daemon: Add `metric_family_compare` and `label_set_compare`.

Merge pull request #4191 from octo/6/load

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

src/daemon/metric.h: Improve doc comment for `metric_family_append`.

./contrib/format.sh src/load.c

Load plugin: Simplify libstatgrab implementation.

Load plugin: rewrite Linux implementation using `read_text_file_contents`.

load plugin: Migrate to collectd 6 data structures.

Merge pull request #4192 from octo/6/irq

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

irq plugin: Remove special case for the "FIQ" interrupt.

format_json: Improve the doc comment for `irq_strsplit`.

Contrast the behavior to `strsplit`.

Co-authored-by: Florian Forster <octo@collectd.org>

irq plugin: Parse counters as `DS_TYPE_DERIVE`.

irq plugin: Document the return value of `irq_strsplit`.

common: Improve coding style of `strsplit`.

* Combine declaration and initialization.

irq plugin: refactor the NetBSD code.

* Declare local variables late.
* Initialize when declaring variables.
* Use `metric_family_append`.

irq plugin: Add additional labels to the Linux implementation.

The Linux implementation will now emit the following fields if available:

* device
* trigger
* kernel_module
* name (a human readable name, e.g. "Non-maskable interrupts")

[collectd 6] irq: migration to v6.0

Merge pull request #4187 from octo/6/resource

[collectd 6] RFC: Add resource attributes.

Move resource attributes into `metric_family_t`.

src/daemon/metric.h: Fix typos in doc comment.

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

Change the semantic of `label_set_clone`.

It checks to ensure that `dest` is empty and returns `EINVAL` otherwise.

src/daemon/metric.c: improve readability of "first_label" logic.

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

src/daemon/metric.h: Fix typo.

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

format_graphite: Add support for resource attributes.

Add resource labels to `metric_t`.

Merge pull request #4176 from octo/6/write_http

[collectd 6] add reference counting to plugins that register multiple callbacks with the same `user_data_t`.

write_riemann plugin: Unify reference counting code with other plugins.

write_http plugin: Unify coding style.

A few plugins implement reference counting. Ensure that they use locks
etc. consistently.

write_tsdb plugin: Implement reference counting.

write_syslog plugin: Implement reference counting.

write_riemann plugin: use reference counting to when freeing user data.

While reference counting was present previously, it had problems:

*   The reference passed to `plugin_register_flush()` was not counted.
*   If a flush plugin was registered, `free_func` was set to NULL but
    the reference passed to `plugin_register_notification()` was still
    counted, meaning in that case the counter never went to zero.
*   Mutexes must be unlocked when calling `pthread_mutex_destroy()`.
*   The code limped on after an error, returning a failure eventually.
    This is unnecessarily complex control flow that has been simplified.

write_stackdriver plugin: Implement reference counting.

src/write_http.c: use reference counting to decide when to free user_data

The teardown code for the wh_callback_t struct previously relied on
the order in which the different callback functions are de-initialized
to be known and to never change.

This is prone to failure and is indeed currently broken, leading to a
segmentation fault on collectd exit.

Fix this by counting the active references to the user data and freeing
it once it reaches zero.

Signed-off-by: Leonard Göhrs <l.goehrs@pengutronix.de>

Merge pull request #4182 from octo/6/configure-write_http

configure: Disable the write_http plugin if libyajl2 is not found.

configure: Disable the write_http plugin if libyajl2 is not found.

Merge pull request #4181 from octo/6/otep119

Update standard system metric names in accordance with OpenTelemetry.

memory plugin: Use standard system metric names (OTEP #119)

https://github.com/open-telemetry/oteps/blob/main/text/0119-standard-system-metrics.md

disk plugin: Use standard system metric names (OTEP #119)

https://github.com/open-telemetry/oteps/blob/main/text/0119-standard-system-metrics.md