write_http plugin: Remove `WH_FORMAT_OTLP_PROTO`.

format_json: Add the `unit` field to the OpenTelemetry output.

collectd.conf(5): Improve wording.

collectd.conf(5): spell "time series database" consistently.

write_http plugin: Adapt to changed resource_metrics semantic.

write_http plugin: Clear the response buffer before issuing the request.

Previously all responses were concatenated together until the buffer was
full, at which point no additional responses were accessible.

write_http plugin: Make the OTLP_JSON output configurable.

Also updates the manpage to reflect this change.

write_http plugin: Add OTLP JSON support using resource metrics.

format_json: Fix unit test.

format_json: Emit only one `ScopeMetrics` per `ResourceMetrics`.

Also fixed the unit test to pass again.

format_json: Update `format_json_open_telemetry` to use resource metrics.

format_json: Move conditional error logging to a separate function.

This reduces the amount of duplicate code being generated.

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

format_json: Use `yajl_gen_double` to encode gauge metrics.

This also fixes the unit test by embedding PACKAGE_VERSION.

format_json: Add a missing "array open" call.

format_json: Add a unit test for the OpenTelemetry format.

format_json: add special error handling to `json_add_string`.

format_json: only log errors when a `yajl_gen_` call fails.

write_http plugin: Add initial support for exporting OTLP via HTTP.

format_json: Add `format_json_open_telemetry` for creating OpenTelemetry JSON.

Merge pull request #4219 from octo/6/memory

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

memory plugin: Align metrics with OpenTelemetry recommendations.

* The "state" label has been renamed to "system.memory.state".
* Metric descriptions and units have been added.

Merge pull request #4220 from octo/6/write_prometheus

write_prometheus plugin: Use the `unit` field to create metric names.

write_prometheus plugin: Remove invalid characers from the unit.

contrib/format.sh src/write_prometheus.c

write_prometheus plugin: Use the `unit` field to create metric names.

Reference:
https://opentelemetry.io/docs/specs/otel/compatibility/prometheus_and_openmetrics/#metric-metadata-1

Merge pull request #4194 from octo/6/unit

[collectd 6] Add a `unit` field to the `metric_family_t` struct.

contrib/format.sh src/daemon/unit.h

src/daemon/unit.h: Reference OpenTelemetry docs.

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

src/daemon/unit.c: Use `string_has_suffix()` from `common`.

src/daemon/unit.c: Add reference link to metric names and units.

src/daemon/metric.h: Add doc comment for the `unit` field.

src/daemon/plugin.c: Automatically determine the unit if possible.

src/daemon/unit.[ch]: Add utility for determining the unit.

src/daemon/metric.[ch]: Add `unit` to `metric_family_t`.

Merge pull request #4212 from octo/6/names

[collectd 6] Allow arbitrary UTF-8 strings as label names.

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.