clang-format: re-format for clang-18

clang-format-18 isn't compatible with the clang-format-17 output, so we
need to reformat the code with the latest version. The way clang-18
handles lambda forma

clang-format: re-format for clang-18

clang-format-18 isn't compatible with the clang-format-17 output, so we
need to reformat the code with the latest version. The way clang-18
handles lambda formatting also changed, so we have made changes to the
organization default style format to better handle lambda formatting.

See I5e08687e696dd240402a2780158664b7113def0e for updated style.
See Iea0776aaa7edd483fa395e23de25ebf5a6288f71 for clang-18 enablement.

Change-Id: If87740ef153661e338448c9de753cbd61fd164eb
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

storage metric: add generic implementation

Add the generic implementation for the storage metric, so there is no
need to add a new enum value and new code, every time a new storage
type needs to be

storage metric: add generic implementation

Add the generic implementation for the storage metric, so there is no
need to add a new enum value and new code, every time a new storage
type needs to be added.

Change-Id: Idcd99455ca559eb6274a71613684a99b4b63a6f0
Signed-off-by: Jagpal Singh Gill <paligill@gmail.com>

show more ...

threshold: use absolute value for D-Bus

Use the absolute value for threshold rather than percentage value.
This involves passing the total value from the metric collection so it
can be used in perce

threshold: use absolute value for D-Bus

Use the absolute value for threshold rather than percentage value.
This involves passing the total value from the metric collection so it
can be used in percentage calculation within health metric logic.

Change-Id: I46240aee54117c70637f9fbf882018f0eac520b2
Signed-off-by: Jagpal Singh Gill <paligill@gmail.com>

show more ...

metric-config: remove the buffer/cache-memory threshold

The kernel will naturally attempt to use almost all free memory up with
buffers and caches. We should not have a threshold on this behavior
b

metric-config: remove the buffer/cache-memory threshold

The kernel will naturally attempt to use almost all free memory up with
buffers and caches. We should not have a threshold on this behavior
by default.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ic40a260667e89d62820e99753efda025ba5bde47

show more ...

metric-config: remove the free-memory threshold

By default we have a metric on the available memory. Free memory should
be a subset of available memory. You can have some memory held by
kernel cac

metric-config: remove the free-memory threshold

By default we have a metric on the available memory. Free memory should
be a subset of available memory. You can have some memory held by
kernel caches that can be evicted as needed, which show up in available
and not free. Having a threshold on free memory is probably not what
anyone wants by default, so remove it from the default config.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I16d15266b6e10e6c7a788095b4b29d5e603e102c

show more ...

metric-config: remove thresholds on userspace and kernel CPU

We do not need a default threshold on the userspace and kernel CPU
metrics because we already have a threshold on the overall CPU usage.

metric-config: remove thresholds on userspace and kernel CPU

We do not need a default threshold on the userspace and kernel CPU
metrics because we already have a threshold on the overall CPU usage.
Sustained CPU usage is likely what someone wants to identify, no matter
which side of the accounting it is attributed to. Delete the extra
thresholds.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I56352169b3fc50f11410f054053c39c40a79b82e

show more ...

metric_config: add lower bound support

Add the lower bound support to the metric config and fix the parsing
logic to update threshold configs accordingly. Update the collection
test for lower bound.

metric_config: add lower bound support

Add the lower bound support to the metric config and fix the parsing
logic to update threshold configs accordingly. Update the collection
test for lower bound.

Change-Id: I6fd3bd8d85974f95ba877eb8d19a77c8acffef7c
Signed-off-by: Jagpal Singh Gill <paligill@gmail.com>

show more ...

sdbusplus: use shorter type aliases

The sdbusplus headers provide shortened aliases for many types.
Switch to using them to provide better code clarity and shorter
lines. Possible replacements are

sdbusplus: use shorter type aliases

The sdbusplus headers provide shortened aliases for many types.
Switch to using them to provide better code clarity and shorter
lines. Possible replacements are for:
* bus_t
* exception_t
* manager_t
* match_t
* message_t
* object_t
* slot_t

Change-Id: I7948c1c8108da238952abaa3e7d400fdca80c7e5
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

add health_metric_collection implementation

Add the interface and implementation for health_metric_collection which
encapsulates various health_metrics.
This change is in relation to following desig

add health_metric_collection implementation

Add the interface and implementation for health_metric_collection which
encapsulates various health_metrics.
This change is in relation to following design and D-Bus interface
update -
https://gerrit.openbmc.org/c/openbmc/docs/+/64917
https://gerrit.openbmc.org/c/openbmc/phosphor-dbus-interfaces/+/64914

Add test_health_metric_collection gtest for unit testing.

Change-Id: Ia0b9fbc6bec4850735c7eb74dcd5c40fc47c568c
Signed-off-by: Jagpal Singh Gill <paligill@gmail.com>

show more ...