report_manager: use PDI constants

Use PDI constants for interface Telemetry.ReportManager

Tested: Inspection only.

Change-Id: I1ae01de009038c142bced22a94b260aa013bee05
Signed-off-by: Alexander Han

report_manager: use PDI constants

Use PDI constants for interface Telemetry.ReportManager

Tested: Inspection only.

Change-Id: I1ae01de009038c142bced22a94b260aa013bee05
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

show more ...

tests: DBus sensor object: use PDI constants

Tested: Inspection only

Change-Id: I1c8f865fc6e7677e0ad59296425fb15c2ce0e848
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

trigger_manager: use PDI constants

Tested: Inspection only

Change-Id: I1ef7abbf7f0741e832395a11002568b1d8ba5bdc
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

trigger: use PDI constants

Use PDI constants for
- Telemetry.Trigger
- Object.Delete

Tested: Inspection only

Change-Id: I15cf1aa35b9266d1ed6ae6b179eb5fd7b4e1b9e9
Signed-off-by: Alexander Hansen <a

trigger: use PDI constants

Use PDI constants for
- Telemetry.Trigger
- Object.Delete

Tested: Inspection only

Change-Id: I15cf1aa35b9266d1ed6ae6b179eb5fd7b4e1b9e9
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

show more ...

sensor: use PDI constants for Sensor.Value

Tested: Inspection only.

Change-Id: I7932f8586701d4e17ad037d14d51b556c3308778
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

report: use PDI constants

use PDI constants for

- Telemetry.Report
- Object.Delete

Tested: Inspection only.

Change-Id: Ieb0a61bd80ceaf7739963024de23eefbe8b2c266
Signed-off-by: Alexander Hansen <a

report: use PDI constants

use PDI constants for

- Telemetry.Report
- Object.Delete

Tested: Inspection only.

Change-Id: Ieb0a61bd80ceaf7739963024de23eefbe8b2c266
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

show more ...

dbus_mapper: use PDI constants

Tested: Inspection only.

Change-Id: Iefeb1f8a1e4a95b985ecae73d2e8fa50f2f9f674
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

cleanup README.md

Remove links to meson documentation. Most OpenBMC projects are meson
projects and the build process does not need documenting at repo level
for the generic meson commands.

Remove

cleanup README.md

Remove links to meson documentation. Most OpenBMC projects are meson
projects and the build process does not need documenting at repo level
for the generic meson commands.

Remove notes / workarounds for boost dependency.
Usage of boost dependency is widespread and does not need special
documentation at the repo level, unless we are doing something special.
When boost dependency is missing on a system, meson can just use a
subproject, no need for manual steps or env variables.

Remove links to general project documentation.

Keep all links which are specific to this repo and it's
design / implementation.

Change-Id: I9cc3c33ce322c4cf452f15eb88a33b17c611c370
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

show more ...

clang-tidy: enable bugprone-unchecked-optional-ac

Add bugprone-unchecked-optional-access check to detect unsafe
std::optional access without prior value checks.

Change-Id: I0bc1d1a2c00fd926ee1da511

clang-tidy: enable bugprone-unchecked-optional-ac

Add bugprone-unchecked-optional-access check to detect unsafe
std::optional access without prior value checks.

Change-Id: I0bc1d1a2c00fd926ee1da5115fdcb386a71f6a1a
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: guard optional JSON access

Add explicit control flow before dereferencing
std::optional<nlohmann::json> in src/report_manager.cpp. Read fields
only on the engaged path to satisfy bugpron

clang-tidy: guard optional JSON access

Add explicit control flow before dereferencing
std::optional<nlohmann::json> in src/report_manager.cpp. Read fields
only on the engaged path to satisfy bugprone-unchecked-optional-access
and keep builds passing with -Werror. No functional changes.

Fixed below type errors

'''
../src/report_manager.cpp:186:30: error: unchecked access to optional value [bugprone-unchecked-optional-access,-warnings-as-errors]
186 | size_t version = data->at("Version").get<size_t>();
| ^~~~
../src/report_manager.cpp:191:28: error: unchecked access to optional value [bugprone-unchecked-optional-access,-warnings-as-errors]
191 | bool enabled = data->at("Enabled").get<bool>();
| ^~~~
../src/report_manager.cpp:192:31: error: unchecked access to optional value [bugprone-unchecked-optional-access,-warnings-as-errors]
192 | std::string& id = data->at("Id").get_ref<std::string&>();
| ^~~~
../src/report_manager.cpp:193:33: error: unchecked access to optional value [bugprone-unchecked-optional-access,-warnings-as-errors]
193 | std::string& name = data->at("Name").get_ref<std::string&>();
| ^~~~
'''

Change-Id: I76faad387fa3786b8770653c745c7ea05d525911
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

Remove LogToJournal trigger action

It has been decided to drop support to LogToLogService action on
Redfish, because it doesn't comply with specification in its current
form. Its dbus counterpart is

Remove LogToJournal trigger action

It has been decided to drop support to LogToLogService action on
Redfish, because it doesn't comply with specification in its current
form. Its dbus counterpart is LogToJournal, therefore this change
removes its implementation.

bmcweb patch: [1]
phosphor-dus-interfaces patch: [2]

Tested:
Verified manually that trigger with LogToJournal action cannot be
created. Unit tests passed.

[1] https://gerrit.openbmc.org/c/openbmc/bmcweb/+/84960
[2] https://gerrit.openbmc.org/c/openbmc/phosphor-dbus-interfaces/+/85268

Change-Id: I82989cdc8c5ab7df70f75614ff24700e14efc4e9
Signed-off-by: Michal Orzel <michalx.orzel@intel.com>

show more ...

Update OWNERS file

Swap repository owner to major contributor - Michal.
Add former contributor - Boleslaw - as a designated reviewer.

Change-Id: I9a0818d976ccbbf94e3dcfbc1086c29e217b0484
Signed-off

Update OWNERS file

Swap repository owner to major contributor - Michal.
Add former contributor - Boleslaw - as a designated reviewer.

Change-Id: I9a0818d976ccbbf94e3dcfbc1086c29e217b0484
Signed-off-by: Adrian Ambrożewicz <adrian.ambrozewicz@intel.com>

show more ...

Resolve coverity issues

* Added Rule of Five compliance to the reported classes
* Added explicit default destructors and assignment operators
* Corrected comparison statement

Tested:
UTs passed

Ch

Resolve coverity issues

* Added Rule of Five compliance to the reported classes
* Added explicit default destructors and assignment operators
* Corrected comparison statement

Tested:
UTs passed

Change-Id: Ifcfc1055e2e65d9f3ce10e3878ff3f1dd5d84929
Signed-off-by: Piotr Sulewski <piotrx.sulewski@intel.com>

show more ...

use sdbusplus unpack syntax

Rather than defining a variable and then reading it from a message,
sdbusplus also supports directly unpack-ing from the message. Use
this syntax instead as it is more e

use sdbusplus unpack syntax

Rather than defining a variable and then reading it from a message,
sdbusplus also supports directly unpack-ing from the message. Use
this syntax instead as it is more efficient and succinct.

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

show more ...

Remove deprecated "Thresholds" property and fix UTs

Thresholds have been split into "DiscreteThresholds" and
"NumericThresholds", remove deprecated property, update UTs

Tested:
UTs passed

Change-I

Remove deprecated "Thresholds" property and fix UTs

Thresholds have been split into "DiscreteThresholds" and
"NumericThresholds", remove deprecated property, update UTs

Tested:
UTs passed

Change-Id: I1959be879e9016b84fff7f7ae6cfad1f5d00e5aa
Signed-off-by: Boleslaw Ogonczyk Makowski <boleslawx.ogonczyk-makowski@intel.com>

show more ...

Fix trigger type for OnChange triggers

Triggers with no threshold parameters should be treated as ones that
activate on change of monitored metric values', with `Discrete` property
value set to true

Fix trigger type for OnChange triggers

Triggers with no threshold parameters should be treated as ones that
activate on change of monitored metric values', with `Discrete` property
value set to true. Otherwise they will be wrongly treated as numeric
triggers on Redfish side. This patch corrects this behavior.

Tested:
1. Created simple discrete trigger: `TestTriggerDiscreteOnChange`, with
`DiscreteTriggerCondition` set to `Changed`.
2. busctl introspect xyz.openbmc_project.Telemetry
/xyz/openbmc_project/Telemetry/Triggers/TelemetryService/
TestTriggerDiscreteOnChange
3. Verified that `Discrete` property is true

All unit tests have passed.

Change-Id: I06762c94b38a9da42a64c9a014d3f6dd47577176
Signed-off-by: Michal Orzel <michalx.orzel@intel.com>

show more ...

Update OWNERS

Change-Id: I3b9fde13549fb87b375f1afbd5bf8c04bc646cae
Signed-off-by: Adrian Ambrożewicz <adrian.ambrozewicz@intel.com>

meson: use non-deprecated systemd packageconfig

Systemd's packageconfig file has both `systemdsystemunitdir` and
`systemd_system_unit_dir` defined. The non-underscore one appears
to be a deprecated

meson: use non-deprecated systemd packageconfig

Systemd's packageconfig file has both `systemdsystemunitdir` and
`systemd_system_unit_dir` defined. The non-underscore one appears
to be a deprecated alias[1]. Move to the non-deprecated /
underscore-separated variant.

[1]: https://github.com/systemd/systemd/commit/4908de44b0a0409f84a7cdc5641b114d6ce8ba03

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

show more ...

clang-tidy: Initial commit

Added initial .clang-tidy configuration file with recommended C++ checks
aligned with the OpenBMC-approved checklist. This setup enforces modern
C++ best practices and hel

clang-tidy: Initial commit

Added initial .clang-tidy configuration file with recommended C++ checks
aligned with the OpenBMC-approved checklist. This setup enforces modern
C++ best practices and helps catch common issues early in development.

Tested: verified build.

Change-Id: I7914c72f734f164793f04df1ca452e44fe3c9617
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: mark unused ioc field in test stub

The 'ioc' field in DbusSensorObject test stub is intentionally
unused but retained to align with the base implementation.

Marked it with [[maybe_unuse

clang-tidy: mark unused ioc field in test stub

The 'ioc' field in DbusSensorObject test stub is intentionally
unused but retained to align with the base implementation.

Marked it with [[maybe_unused]] to suppress clang-tidy's
-Wunused-private-field warning while preserving interface
consistency for future compatibility.

Error details:
'''
tests/src/stubs/dbus_sensor_object.cpp:17:5: error:
class ‘stubs::DbusSensorObject’ does not have any field named ‘ioc’
17 | ioc(ioc), bus(bus), objServer(objServer)
'''

Change-Id: Id86a671b255f98ea22fddca82c24ad2192727235
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: Improve const-correctness

Updated member functions to better reflect const-correctness,
enhancing code clarity and maintainability.

Change-Id: Ib3fdbd2164cb76ad7f0c454d37dd5c08652f78fc

clang-tidy: Improve const-correctness

Updated member functions to better reflect const-correctness,
enhancing code clarity and maintainability.

Change-Id: Ib3fdbd2164cb76ad7f0c454d37dd5c08652f78fc
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: Remove unused lambda captures

The following errors were reported during clang-tidy enablement due
to unused lambda captures.

'''
../src/metric.cpp:119:50: error: lambda capture 'this' i

clang-tidy: Remove unused lambda captures

The following errors were reported during clang-tidy enablement due
to unused lambda captures.

'''
../src/metric.cpp:119:50: error: lambda capture 'this' is not used [-Werror
'''

The fix involves removing these unused lambda captures.

Tested: Build and unit testing verified.

Change-Id: I6a4b42ea3df7513e1208638f427007680e2a15cf
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

fix random UT fails

Tests TestNumericThresholdWithDwellTime/SleepAfterEveryUpdate/2, 6 and 9
could fail randomly because next update happened at the same time when
timer from one of previous updates

fix random UT fails

Tests TestNumericThresholdWithDwellTime/SleepAfterEveryUpdate/2, 6 and 9
could fail randomly because next update happened at the same time when
timer from one of previous updates was supposed to expire - if update
got processed first, timer was canceled and test failed

Tested:
Before this change UTs failed almost 100% of the time on my machine,
with this change 0 failures in 500 iterations

Change-Id: I11940027fca377b7be67cb82b638bf162616d37e
Signed-off-by: Boleslaw Ogonczyk Makowski <boleslawx.ogonczyk-makowski@intel.com>

show more ...

clang-format: update latest spec and reformat

Copy the latest format file from the docs repository and apply.

Change-Id: Ic2bc1d98d8ad469a3477f9059ec890fc8339907d
Signed-off-by: Patrick Williams <p

clang-format: update latest spec and reformat

Copy the latest format file from the docs repository and apply.

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

show more ...

meson: reformat with meson formatter

Apply the `meson format` results.

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