Fix deprecated actions/upload-artifact@v3 in GitHub Actions

(cherry picked from commit 0edf8505c239b021dead7b1226b71db131cae692)

Format

Fix lcc_receive

The unit has been substracted to pass gcc stringop-truncation warning in
lcc_receive function

Fix history accumulating in utils_cache

gpu_sysman: remove  _total suffix for monotonic metrics

As "write_prometheus" plugin already adds that unconditionally to
names of all monotonic metric types.

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

gpu_sysman: usecs/ujoules -> seconds/joules metric types

With the new floating point counter type, joules & seconds can be used
as metric units instead of microjoules & microseconds.

(Frequencies are are still left as Mhz, but for convenience, and
compability with Intel XPU Manager.)

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

gpu_sysman: use new up/down types for cumulative metrics

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

gpu_sysman: use counter_diff() to handle counter overflow

Assumes overflowing counters to be either at 32-bit or 64-bit,
which should cover most common cases, as long as overflow happens
at max once during query period.

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

gpu_sysman: fix ras_submit() argument type

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

Merge pull request #4293 from eero-t/xe-kmd-support

[collectd 6] gpu_sysman: switch from zeInit() to zesInit()

gpu_sysman: use glob.h/libgen.h only if they are present

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

gpu_sysman: add "dev_name" + get "pci_dev" label value from sysfs

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

gpu_sysman: set legacy env var for older backends

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

configure.ac: update gpu_sysman plugin checks for zesInit()

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

gpu_sysman: minor tweaks for GPU info output

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

gpu_sysman: drop L0 core items not supported with zesInit()

When just Sysman is initialized (by using zesInit() instead of
zeInit()), L0 core functions are no-ops and core structs are not
filled (except for UUID), so drop those.

Try also to present the datata in slightly more readable format
by re-ordering it a bit and making indentation more consistent.

As Sysman does not provide device PCI ID for "pci_dev" label,
take device (model/marketing) name instead, and assing it to
"dev_name" label (like Intel XPU Manager already does).

Changing the "pci_dev" member name, is left for later, as it
would conflict with resource attribute and OpenTelemetry metric
name work being done in parallel.

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

gpu_sysman: show also mem health + allocatable amount at init

Sysman may return zero for physical memory size, but allocatable size
should always be valid.  This can be useful in identifying different
GPU types (on same machine) from each other.

Skip showing of invalid memory information.

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

gpu_sysman: change from L0 core to Sysman memory APIs

In the GPU info logging function, as L0 core functions do nothing when
(first) init call (zesInit) is done for Sysman.

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

gpu_sysman: zeInit() -> zesInit(), to support Xe GPU KMD

zesInit() requires Level-Zero frontend v1.9 implementing spec v1.5.

Intel L0 Sysman backend supports only "i915" GPU KMD uAPI variants
when initialized using L0 core zeInit() function. Support for the new
"xe" GPU KMD uAPI requires backend to be initialized using (new)
Sysman-specific zesInit() function instead.

Use of zesInit() means that called driver and device functions also
need to be be L0 Sysman versions instead L0 core versions (skipping L0
core initialization may speed plugin initialization).

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

Merge pull request #4177 from eero-t/log-metrics-option

[Collectd 6] gpu_sysman: add "LogMetrics" option

gpu_sysman: WA for local & CI clang-format conflict

Comment prevents local clang-format from trying to re-order
includes differently from the clang-format in CI.

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

gpu_sysman: support logging all (new) metric types

New types are not yet used by the plugin, but soon are.

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

gpu_sysman: skip common metric prefix in metrics logging

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

gpu_sysman: document "LogMetrics" option

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

gpu_sysman: Add "LogMetrics" option

This can be used for local real-time monitoring of GPU metrics without
need to pull/parse collectd write plugin output.

Output is most readable when only one metric type and MetricsOutput
variant are enabled, and collectd sees only single GPU.  Alternatively
one could grep the output for the relevant GPU, metric type and its
output variant.

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

Merge pull request #4288 from octo/6/release

ci: Trigger a website update after uploading a release artifact.

Merge pull request #4273 from octo/6/processes

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

processes plugin: Convert appropriate metrics to "up_down".

processes plugin: Rename "fpcounter" to "counter_fp".

processes plugin: Fix reference of a non-existent variable.

processes plugin: Consistently call `procstat_t*` variables "ps".

processes plugin: Unify the usage of `read_text_file_contents()`.

processes plugin: Add a comment to point readers at more documentation.

processes plugin: Remove obsolete code.

processes plugin: Fix incorrect scaling of `process.cpu.utilization`.

processes plugin: Remove unnecessarily verbose assertion.

processes plugin: Return boolean from `process_resource()`.

processes plugin: Remove unused struct field.

processes plugin: Fix assignment of CPU time counters.

processes plugin: Remove unused fields from `struct procstat`.

processes plugin: Report delay accounting in seconds.

processes plugin: Report context switches as metric family.

processes plugin: Report the fork rate as metric family.

processes plugin: Report process CPU time in seconds.

processes plugin: Port CPU usage to `metric_family_t`.

processes plugin: Use `sysconf(_SC_CLK_TCK)` instead of `CONFIG_HZ`.

processes plugin: Add note about AIX using `struct timeval` weirdly.

processes plugin: Port the delay accounting to `metric_family_t`.

processes plugin: Port metrics to new data structure:

* I/O bytes
* I/O operations
* number of file descriptors
* number of page faults

processes plugin: Initial working code for collectd 6.

configure.ac: Enable the processes plugin by default.

processes plugin: Convert the aggregate system metric.

processes plugin: Break the enormous `ps_read` function into OS specific parts.

ci: Trigger a website update after uploading a release artifact.

Update ChangeLog for version 6.0.0.rc3.

Merge pull request #4287 from octo/6/up_down_counter

[collectd 6] Implement "UpDownCounter" metric types.

memory plugin: Remove unnecessary casts.

common: Improve error case description.

open_telemetry plugin: Receiver: Add support for counter_fp, up_down, and up_down_fp.

Tree wide: Remove `fpcounter_t` and use `double` instead.

Tree wide: Remove `up_down_counter_fp_t` and use `double` instead.

Tree wide: Remove `up_down_counter_t` and use `int64_t` instead.

Tree wide: Rename "fpcounter" to "counter_fp".

Tree wide: Rename "up_down_counter_fp" to "up_down_fp".

Tree wide: Rename "up_down_counter" to "up_down".

contrib/format.sh <all the things>

df plugin: Migrate to "up down counter".

memory plugin: Migrate to "up down counter".

Daemon: add "up down counters" in both an integer and floating point variant.

Merge pull request #4284 from octo/6/write_prometheus

[collectd 6] write_prometheus plugin: Fix handling of multiple resources.

write_prometheus plugin: Unconditionally print the job and instance labels.

write_prometheus plugin: Update unit tests to the new semantics.

write_prometheus plugin: Let "job" default to hostname_g, and "instance" to an empty string.

contrib/format.sh src/write_prometheus.c

write_prometheus plugin: Fix updating of existing metrics (+test).

Merge pull request #4223 from eero-t/sysman-test-metrics

[collectd 6] Use real collectd metric functions in gpu_sysman plugin tests

write_prometheus plugin: Fix formatting in the unit test.

write_prometheus plugin: Simplify files included by the test.

write_prometheus plugin: Complete error handling in `prom_metric_family_clone()`.

write_prometheus plugin: Make all functions `static` again.

Revert "Daemon: Add `metric_list_append`."

This reverts commit 0b3a481b457038b048c19df2af656a925644273f.

contrib/format.sh src/write_prometheus.c src/write_prometheus_test.c

write_prometheus plugin: Refactor the internal data structures.

(Ab)Using the `metric_t` and `metric_family_t` data structures provided by the
daemon made memory management quite hard and therefore brittle.

This introduces types specific to the *write_prometheus plugin* that store the
resource attributes with the metric, not the family.

write_prometheus plugin: Store resource attributes with the metric, not the metric family.

For Prometheus output, the plugin groups all metrics with the same name into
one `metric_family_t`. This caused problems when collectd handled metrics from
multiple resources.

To solve this issue, we're somewhat abusing the data structure and store
per-metric resource attributes in the `family` field. That means for the
metrics stored in the *write_prometheus plugin* `(metric_t).family` does not
point back to the metric family containing the metric.

Fixes: #4283

Daemon: Add `metric_list_append`.

write_prometheus plugin: Add an end-to-end test.

Merge pull request #4271 from octo/6/otelcol

[collectd 6] New plugin (sort of): OpenTelemetry receiver

open_telemetry plugin: Move prototypes to a header file.

open_telemetry: Unify the configuration handling between exporter and receiver.

open_telemetry plugin: Minor cleanups.

* Set field to `NULL` after freeing.
* Remove unused global variable.

open_telemetry plugin: Fix spelling in the manpage.

open_telemetry plugin: Add support for floating point counters.

open_telemetry plugin: Update the synopsis in the manpage.

open_telemetry plugin: Add SSL options for the exporter.

open_telemetry plugin: Unify the configuration between exporter and receiver.

open_telemetry plugin: Unify the prefix used in log messages.

open_telemetry plugin: Combine the *open_telemetry_collector* and *write_open_telemetry* plugins.

open_telemetry_collector plugin: Enable gRPC reflection if available.

open_telemetry_collector plugin: Populate `ssl_opts` only when needed.

open_telemetry_collector plugin: Link with protobuf library.