commit 56ccc249f3f45bcb7486c93a19ab271173c64c68
author Denis Zlobin <zlobin.d.al@gmail.com> Thu Apr 27 07:29:03 2023 +0000
committer Denis Zlobin <zlobin.d.al@gmail.com> Mon May 22 12:21:37 2023 +0000
tree 3041f15f7f688b3ef404429fa8bef4506877354e
parent 6f5342d0bdb2b2b8c52323219656f88794abf54f

Add pulse sensor log events to logging service

Add logging for host power on/off events with SEL_LOGGER_SEND_TO_LOGGING_SERVICE
enabled.

Tested: Check current log entries by using busctl command:
busctl tree xyz.openbmc_project.Logging
Turn on host power. Check new log entry message:
busctl get-property xyz.openbmc_project.Logging /xyz/openbmc_project/logging/entry/<id> xyz.openbmc_project.Logging.Entry Message
Message must contain text:
"Host system DC power is on"
Turn off host power. Check new log entry message. Message must contain text:
"Host system DC power is off"

Change-Id: I1f7339182f82cbf4509e34034d09d05e66ca2948
Signed-off-by: Denis Zlobin <zlobin.d.al@gmail.com>

include/pulse_event_monitor.hpp

diff --git a/include/pulse_event_monitor.hpp b/include/pulse_event_monitor.hpp
index 3d53720..83b0632 100644
--- a/include/pulse_event_monitor.hpp
+++ b/include/pulse_event_monitor.hpp
@@ -22,7 +22,7 @@
 inline static sdbusplus::bus::match_t
     startPulseEventMonitor(std::shared_ptr<sdbusplus::asio::connection> conn)
 {
-    auto pulseEventMatcherCallback = [](sdbusplus::message_t& msg) {
+    auto pulseEventMatcherCallback = [conn](sdbusplus::message_t& msg) {
         std::string thresholdInterface;
         boost::container::flat_map<std::string, std::variant<std::string>>
             propertiesChanged;
@@ -45,25 +45,43 @@
 
         if (event == "CurrentHostState")
         {
+
+            std::string journalMsg;
+            std::string redfishMsgId;
+
             if (*variant == "xyz.openbmc_project.State.Host.HostState.Off")
             {
-                std::string message("Host system DC power is off");
-                std::string redfishMsgId("OpenBMC.0.1.DCPowerOff");
-
-                sd_journal_send("MESSAGE=%s", message.c_str(),
-                                "REDFISH_MESSAGE_ID=%s", redfishMsgId.c_str(),
-                                NULL);
+                journalMsg = "Host system DC power is off";
+                redfishMsgId = "OpenBMC.0.1.DCPowerOff";
             }
             else if (*variant ==
                      "xyz.openbmc_project.State.Host.HostState.Running")
             {
-                std::string message("Host system DC power is on");
-                std::string redfishMsgId("OpenBMC.0.1.DCPowerOn");
-
-                sd_journal_send("MESSAGE=%s", message.c_str(),
-                                "REDFISH_MESSAGE_ID=%s", redfishMsgId.c_str(),
-                                NULL);
+                journalMsg = "Host system DC power is on";
+                redfishMsgId = "OpenBMC.0.1.DCPowerOn";
             }
+            else
+            {
+                return;
+            }
+#ifdef SEL_LOGGER_SEND_TO_LOGGING_SERVICE
+            sdbusplus::message_t newLogEntry = conn->new_method_call(
+                "xyz.openbmc_project.Logging", "/xyz/openbmc_project/logging",
+                "xyz.openbmc_project.Logging.Create", "Create");
+            const std::string logLevel =
+                "xyz.openbmc_project.Logging.Entry.Level.Informational";
+            const std::string hostPathName = "HOST_PATH";
+            const std::string hostPath = msg.get_path();
+            newLogEntry.append(
+                std::move(journalMsg), std::move(logLevel),
+                std::map<std::string, std::string>(
+                    {{std::move(hostPathName), std::move(hostPath)}}));
+            conn->call(newLogEntry);
+#else
+            sd_journal_send("MESSAGE=%s", journalMsg.c_str(),
+                            "REDFISH_MESSAGE_ID=%s", redfishMsgId.c_str(),
+                            NULL);
+#endif
         }
     };