src/daemon/resource.c: Allow default resource attributes to be empty.

To achieve this, track whether or not the struct has been initialized
separately, so that users can remove all the attributes if they wish.

src/collectd.conf.pod: Describe "resources" in more detail.

libplugin_mock: Add `src/daemon/resource.[ch]`.

Since `configfile` now refers to `resource`, this needs to be linked into
tests.

configfile: Avoid allocating a buffer for the resource type.

Mostly to avoid an uninformed compiler warning ;)

resource: Add an option to configure resource attributes.

Merge pull request #4208 from octo/6/built-plugins

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

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