Makefile.am: Fix inconsistent indentation.

Merge branch 'collectd-6.0' into 6/names

Merge pull request #4213 from octo/6/write_prometheus

[collectd 6] Improve Write Prometheus' handling of resource attributes.

write_prometheus plugin: Emit a single `target_info` metric family.

With this change, the *write_promtheus plugin* will emit a single
`target_info` metric family with one metric per resource.

write_prometheus plugin: Insert a newline between metric families.

resource_metrics: Compile into a library.

resource_metrics: Fix an incorrect pointer type.

It's only ever used for a NULL comparison, so it doesn't really matter,
but it is unnecessarily confusing to use an incorrect type.

write_prometheus plugin: Emit the "target info" label as a gauge by default.

Prometheus doesn't know the "info" type metrics that OpenMetrics talks
about. Instead, is has a *convention* of using a gauge metric named
"target_info".

write_prometheus plugin: Improve label sanitation.

This copies the behavior of the OpenTelemetry Prometheus translator:

1.  Replace all invalid characters with underscores.
2.  If the name starts with a single underscore, add the prefix `key`.
3.  If the name stars with a digit, add the prefix `key_`.

src/daemon/metric.[ch]: Export `label_set_get` and `label_set_update`.

write_prometheus plugin: Improve metric family name generation.

write_prometheus plugin: Emit resource attributes only, if they differ from the default.

write_prometheus plugin: Translate between resource attributes and target labels.

* `service.name` → `job`
* `service.instance.id` → `instance`

write_prometheus plugin: Emit special a "target info" metric.

src/daemon/metric_test.c: Add invalid inputs for `metric_parse_identity`.

src/daemon/metric.c: Allow arbitrary UTF-8 strings are label name.

src/daemon/metric_test.c: Add test for `metric_parse_identity`.

utf8: Handle `NULL` inputs gracefully.

utf8: Make `decode` idempotent by returning the new state.

utf8: Remove the unused `codep` argument.

utf8: Iterate over the input string using a for loop.

This is arguably easier to read.

utf8: Make `decode` a void function -- its return value is always ignored.

utf8: Change return value to `bool`.

utf8: Change argument from `uint8_t *` to `char const *`.

utf8: Rename `IsUTF8` to `utf8_valid` to match collectd's naming schema.

utf8: Add unit test.

utf8: Add a header file.

utf8: Make utility build.

*   Import `"collectd.h"` for `<stdint.h>`.
*   Make `decode` static, otherwise GCC complains about it reading from a
    static variable.

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

utf8: Import code from http://bjoern.hoehrmann.de/utf-8/decoder/dfa/.

Co-authored-by: Bjoern Hoehrmann <bjoern@hoehrmann.de>

Merge pull request #4214 from octo/label_set_delete

[collectd 6] src/daemon/metric.c: Fix memory corruption in `label_set_delete`.

src/daemon/metric.[ch]: Export `label_set_format`.

This is now also used in the metric test, replacing the custom
implementation there.

src/daemon/metric.c: Fix memory corruption in `label_set_delete`.

The size parameter passed to `memmove` was in number of pairs, not bytes.

src/daemon/metric_test.c: Test the integrity of `metric_label_set`.

The previous test only checked whether the added/removed value could be
retrieved. Also verify that the remainder of the label set is intact.

Merge pull request #4199 from octo/6/resource

[collectd 6] Add an option to configure resource attributes.

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