commit a2b4c7a174d215048f75ef5c5d8217ad76d1f5e4
author Jason M. Bills <jason.m.bills@linux.intel.com> Mon Jun 24 16:55:21 2019 -0700
committer Vernon Mauery <vernon.mauery@linux.intel.com> Wed Jun 26 22:40:10 2019 +0000
tree 38ab04079bf59c5e10c88e141ae1d3199ae0b3a0
parent b08f84e3b1d6fb140320070ac9eaa2b5f5e478b9

Update AC fail check to new DBus parameter

PFail was removed from DBus and replaced with a new parameter
in settings.  This changes the Get Chassis Status command to
check the new parameter to get the AC fail status.

Tested:
After AC cycle:
ipmitool chassis status
Last Power Event     : ac-failed

Change-Id: I019a48c1347dced5b81866ec3dcd81c5964561fd
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>

src/chassiscommands.cpp

diff --git a/src/chassiscommands.cpp b/src/chassiscommands.cpp
index 2fa9e23..9c37939 100644
--- a/src/chassiscommands.cpp
+++ b/src/chassiscommands.cpp
@@ -292,29 +292,26 @@
  */
 bool getACFailStatus()
 {
-    constexpr const char* powerControlObj =
-        "/xyz/openbmc_project/Chassis/Control/Power0";
-    constexpr const char* powerControlIntf =
-        "xyz.openbmc_project.Chassis.Control.Power";
-    bool acFail = false;
+    constexpr const char* acBootObj =
+        "/xyz/openbmc_project/control/host0/ac_boot";
+    constexpr const char* acBootIntf = "xyz.openbmc_project.Common.ACBoot";
+    std::string acFail;
     std::shared_ptr<sdbusplus::asio::connection> bus = getSdBus();
     try
     {
-        auto service =
-            ipmi::getService(*bus, powerControlIntf, powerControlObj);
+        auto service = ipmi::getService(*bus, acBootIntf, acBootObj);
 
-        ipmi::Value variant = ipmi::getDbusProperty(
-            *bus, service, powerControlObj, powerControlIntf, "PFail");
-        acFail = std::get<bool>(variant);
+        ipmi::Value variant = ipmi::getDbusProperty(*bus, service, acBootObj,
+                                                    acBootIntf, "ACBoot");
+        acFail = std::get<std::string>(variant);
     }
     catch (const std::exception& e)
     {
-        log<level::ERR>("Failed to fetch PFail property",
-                        entry("ERROR=%s", e.what()),
-                        entry("PATH=%s", powerControlObj),
-                        entry("INTERFACE=%s", powerControlIntf));
+        log<level::ERR>(
+            "Failed to fetch ACBoot property", entry("ERROR=%s", e.what()),
+            entry("PATH=%s", acBootObj), entry("INTERFACE=%s", acBootIntf));
     }
-    return acFail;
+    return acFail == "True";
 }
 } // namespace power_policy