Move the control application to lg2

Update the logging API used for files in phosphor-fan-presence/control
from the older phosphor::logging::log to the more recent lg2::log.

Tested:
* Verified jour

Move the control application to lg2

Update the logging API used for files in phosphor-fan-presence/control
from the older phosphor::logging::log to the more recent lg2::log.

Tested:
* Verified journal traces worked correctly in simulation.
* Modified json to force various property errors and verified
correct key/value pairs in the journalctl output.

Change-Id: If1ac33b90abe2a9baae6f3cc788863bd514fa80e
Signed-off-by: Anwaar Hadi <anwaar.hadi@ibm.com>

show more ...

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: Ica590f8613f1fb89ab1ca676ac51c1cc7e38d67f
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

use std::format instead of fmt::format

The std::format is sufficient for the uses in this repository except
for in one file (override_fan_target.cpp, since P2286 isn't supported
by GCC yet). Switch

use std::format instead of fmt::format

The std::format is sufficient for the uses in this repository except
for in one file (override_fan_target.cpp, since P2286 isn't supported
by GCC yet). Switch to std::format whenever possible.

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

show more ...

clang-format: copy latest and re-format

clang-format-16 has some backwards incompatible changes that require
additional settings for best compatibility and re-running the formatter.
Copy the latest

clang-format: copy latest and re-format

clang-format-16 has some backwards incompatible changes that require
additional settings for best compatibility and re-running the formatter.
Copy the latest .clang-format from the docs repository and reformat the
repository.

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

show more ...

meson support: remove code warnings 2

This commit contains code changes necessary to support the increased
warning level from Meson builds. Most changes are for unused variables.
to keep the review

meson support: remove code warnings 2

This commit contains code changes necessary to support the increased
warning level from Meson builds. Most changes are for unused variables.
to keep the review size manageable, this commit contains only control
changes (plus one in sensor-monitor).

Change-Id: Ie20f1d9028add4b605e4cc9fb230940710365706
Signed-off-by: Mike Capps <mikepcapps@gmail.com>

show more ...

control: Remove two debug traces

There are valid cases where a parameter value may not exist even when it
is used in the set_net_increase/decrease_target event configs, so remove
the debug traces th

control: Remove two debug traces

There are valid cases where a parameter value may not exist even when it
is used in the set_net_increase/decrease_target event configs, so remove
the debug traces that complain when it isn't there so they don't fill up
the journal.

Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I2c7ca6d58fe073f8164e1b9cadae2b44e6ea37e6

show more ...

control: Parameter use for set_net_increase_speed

Change the set_net_increase_speed to be able to specify a Manager
parameter to get the state value from instead of just looking up a
hardcoded value

control: Parameter use for set_net_increase_speed

Change the set_net_increase_speed to be able to specify a Manager
parameter to get the state value from instead of just looking up a
hardcoded value in the JSON.

The JSON now supports a 'state_parameter_name' field that can be used in
place of the 'state' field. The code will then call
Manager::getParameter with this parameter name to get the state value to
use in the action.

Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: Iaf422787c57c60a3a90c3272813e5f4eb3ca9886

show more ...

control: Actions use list of groups set on base object

Remove the reference to a group given when running an action and instead
have each action iterate over the list of groups configured/set on the

control: Actions use list of groups set on base object

Remove the reference to a group given when running an action and instead
have each action iterate over the list of groups configured/set on the
base action object. The group configured/set on the base action object
is the only set of groups that the action should use based on its
configuration in an event.

Change-Id: Ia298d270e782ad729b5a29f9cdfe9b6c9ea4bc45
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>

show more ...

control: Update base action object with zones and groups used

The base action object stores the groups and zones configured for an
action to use and run against. When a trigger fires, it will run th

control: Update base action object with zones and groups used

The base action object stores the groups and zones configured for an
action to use and run against. When a trigger fires, it will run the
action against each zone configured. Each action will have its list of
groups to use that it can iterate over appropriately. This will be
updated in the next commit to no longer pass a reference to a group when
running an action and to have each action iterate over the groups
configured for them to use.

Change-Id: Ifcd1a260e38af87c5f3bb2a3d44813e71743abc6
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>

show more ...

control: Add net target increase action

Add the YAML based set_net_speed_increase action function as an action
class for JSON configs to use.

Change-Id: Ia7e5cb8bcdb4bba0ebf356d9dc774ab292f0e8fc
Si

control: Add net target increase action

Add the YAML based set_net_speed_increase action function as an action
class for JSON configs to use.

Change-Id: Ia7e5cb8bcdb4bba0ebf356d9dc774ab292f0e8fc
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>

show more ...