commit 5571acd2359e4420d749e14069e3eefe1f5b125b
author Peter Yin <peter.yin@quantatw.com> Tue Jun 17 18:02:20 2025 +0800
committer Ed Tanous <ed@tanous.net> Tue Jun 24 14:48:09 2025 +0000
tree 8d01f9999f50ed46d62c53799b8ccee30825770d
parent e2cdf06fe56aeb12e6c76e494487b1f1e93d474a

redfish: manager: skip AccumulateSetPoint in zone config parsing

AccumulateSetPoint is a boolean type, but the current parsing logic
for the xyz.openbmc_project.Configuration.Pid.Zone interface assumes
all properties are of type double. This mismatch causes an internal
error when processing AccumulateSetPoint

Fixed
bmcwebd: [openbmc_managers.hpp:284] Field Illegal AccumulateSetPoint
bmcwebd: [error_messages.cpp:1250] Internal Error /usr/src/debug/
bmcweb/1.0+git/redfish-core/lib/openbmc/openbmc_managers.hpp(285:56)
`redfish::asyncPopulatePid(const std::string&, const std::string&,
const std::string&, const std::vector<std::__cxx11::basic_string<char>,
std::allocator<std::__cxx11::basic_string<char> > >&,
const std::shared_ptr<bmcweb::AsyncResp>&)::<lambda
(const boost::system::error_code&, const dbus::utility::
ManagedObjectType&)>`:

Change-Id: Idd11afa067fd20e047a28f72307231e2b76bc593
Signed-off-by: Peter Yin <peter.yin@quantatw.com>

redfish-core/lib/openbmc/openbmc_managers.hpp

diff --git a/redfish-core/lib/openbmc/openbmc_managers.hpp b/redfish-core/lib/openbmc/openbmc_managers.hpp
index b8bacc3..6bd52e5 100644
--- a/redfish-core/lib/openbmc/openbmc_managers.hpp
+++ b/redfish-core/lib/openbmc/openbmc_managers.hpp
@@ -268,7 +268,8 @@
                     {
                         if (propertyPair.first == "Type" ||
                             propertyPair.first == "Class" ||
-                            propertyPair.first == "Name")
+                            propertyPair.first == "Name" ||
+                            propertyPair.first == "AccumulateSetPoint")
                         {
                             continue;
                         }