commit be14ec2d4350ba245538ad0330587af809fa31c2
author Matt Spinler <spinler@us.ibm.com> Fri Mar 24 15:59:48 2023 -0500
committer Matt Spinler <spinler@us.ibm.com> Fri Mar 24 16:02:03 2023 -0500
tree 531d703b59607f4d77a06ae4764aaf8658c59741
parent 211f8a9a8ceac7e0d369c9b76c4ff542ccfb8ff0

Change daily clock log to informational severity

The event log was using a severity of Error, and because the PEL message
registry had a severity of Informational, phosphor-log-manager would
spit out a trace about how the severities mismatched every time the PEL
was created.

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

extensions/phal/clock_logger.cpp

diff --git a/extensions/phal/clock_logger.cpp b/extensions/phal/clock_logger.cpp
index 5de6047..6376988 100644
--- a/extensions/phal/clock_logger.cpp
+++ b/extensions/phal/clock_logger.cpp
@@ -159,7 +159,7 @@
     addClockRegData(clockDataLog);
 
     openpower::pel::createPEL("org.open_power.PHAL.Info.ClockDailyLog",
-                              clockDataLog);
+                              clockDataLog, Severity::Informational);
 }
 
 void Manager::addCFAMData(struct pdbg_target* proc,

extensions/phal/create_pel.cpp

diff --git a/extensions/phal/create_pel.cpp b/extensions/phal/create_pel.cpp
index 6cbf305..c834419 100644
--- a/extensions/phal/create_pel.cpp
+++ b/extensions/phal/create_pel.cpp
@@ -239,7 +239,8 @@
     return plid;
 }
 
-void createPEL(const std::string& event, const FFDCData& ffdcData)
+void createPEL(const std::string& event, const FFDCData& ffdcData,
+               const Severity severity)
 {
     std::map<std::string, std::string> additionalData;
     auto bus = sdbusplus::bus::new_default();
@@ -258,8 +259,7 @@
                                           loggingInterface, "Create");
         auto level =
             sdbusplus::xyz::openbmc_project::Logging::server::convertForMessage(
-                sdbusplus::xyz::openbmc_project::Logging::server::Entry::Level::
-                    Error);
+                severity);
         method.append(event, level, additionalData);
         auto resp = bus.call(method);
     }

extensions/phal/create_pel.hpp

diff --git a/extensions/phal/create_pel.hpp b/extensions/phal/create_pel.hpp
index ffd1315..a6256f5 100644
--- a/extensions/phal/create_pel.hpp
+++ b/extensions/phal/create_pel.hpp
@@ -55,8 +55,10 @@
  *
  *  @param[in]  event - the event type
  *  @param[in] ffdcData - failure data to append to PEL
+ *  @param[in] severity - severity of the log
  */
-void createPEL(const std::string& event, const FFDCData& ffdcData = {});
+void createPEL(const std::string& event, const FFDCData& ffdcData = {},
+               const Severity severity = Severity::Error);
 
 /**
  * @class FFDCFile