commit 52e54e875dcd7a76368bb4a70b5e24ff7fcdbcb8
author Brandon Wyman <bjwyman@gmail.com> Thu Oct 08 14:44:58 2020 -0500
committer Brandon Wyman <bjwyman@gmail.com> Tue Oct 13 17:49:20 2020 -0500
tree 3b5c33b5d7a888f60e964dda96177055e1ef81e8
parent b76ab2497e1010c238035b642bfa2123aad66bdf

psu-ng: Log errors for missing power supplies

If a power supply is not present, create an error.

Signed-off-by: Brandon Wyman <bjwyman@gmail.com>
Change-Id: I8848f00572019588facbd44405e8d37d5106fb68

phosphor-power-supply/psu_manager.cpp

diff --git a/phosphor-power-supply/psu_manager.cpp b/phosphor-power-supply/psu_manager.cpp
index c84201e..32847b8 100644
--- a/phosphor-power-supply/psu_manager.cpp
+++ b/phosphor-power-supply/psu_manager.cpp
@@ -176,11 +176,20 @@
 
     for (auto& psu : psus)
     {
+        std::map<std::string, std::string> additionalData;
+        additionalData["_PID"] = std::to_string(getpid());
         // TODO: Fault priorities #918
-        if (!psu->isFaultLogged() && psu->isFaulted())
+        if (!psu->isFaultLogged() && !psu->isPresent())
         {
-            std::map<std::string, std::string> additionalData;
-            additionalData["_PID"] = std::to_string(getpid());
+            // Create error for power supply missing.
+            additionalData["CALLOUT_INVENTORY_PATH"] = psu->getInventoryPath();
+            additionalData["CALLOUT_PRIORITY"] = "H";
+            createError("xyz.openbmc_project.Power.PowerSupply.Error.Missing",
+                        additionalData);
+            psu->setFaultLogged();
+        }
+        else if (!psu->isFaultLogged() && psu->isFaulted())
+        {
             additionalData["STATUS_WORD"] =
                 std::to_string(psu->getStatusWord());