commit | 3a34b742207414e20eb928e982b7c217789e89d5 | [log] [tgz] |
---|---|---|
author | Gunnar Mills <gmills@us.ibm.com> | Fri Jul 28 10:17:14 2023 -0500 |
committer | Ed Tanous <ed@tanous.net> | Fri Jul 28 16:25:50 2023 +0000 |
tree | a929bccbf76f766006647aca937e1da73e2e3749 | |
parent | d3dfb6eba0a4b95eb6630c289fa80be2203861e8 [diff] [blame] |
Fix RestorePolicy map Before this code looked like: {"xyz.openbmc_project.Control.Power.RestorePolicy.Policy.Restore", "LastState"}, and that matches the PDI: https://github.com/openbmc/phosphor-dbus-interfaces/blob/d1484a1499bc241316853934e6e8b735166deee2/yaml/xyz/openbmc_project/Control/Power/RestorePolicy.interface.yaml#L33 https://gerrit.openbmc.org/c/openbmc/bmcweb/+/64334/2/redfish-core/lib/systems.hpp#1350 changed this to RestorePolicy.Policy.AlwaysRestore, I think just a copy mistake. Pointed out by Patrick Rudolph. Tested: None. Searching the enum now matches PDI and what was there before. Change-Id: I90e41d352dd1d1de3e465b4f2d5c9f78e0de5b3b Signed-off-by: Gunnar Mills <gmills@us.ibm.com>
diff --git a/redfish-core/lib/systems.hpp b/redfish-core/lib/systems.hpp index 54dbd70..188ff32 100644 --- a/redfish-core/lib/systems.hpp +++ b/redfish-core/lib/systems.hpp
@@ -1357,7 +1357,7 @@ return computer_system::PowerRestorePolicyTypes::AlwaysOff; } if (value == - "xyz.openbmc_project.Control.Power.RestorePolicy.Policy.AlwaysRestore") + "xyz.openbmc_project.Control.Power.RestorePolicy.Policy.Restore") { return computer_system::PowerRestorePolicyTypes::LastState; }