commit b464c8bd2ef1ecaae07ac0d39fb7dbc1a52fda5f
author Shawn McCarney <shawnmm@us.ibm.com> Thu Jul 16 17:51:38 2020 -0500
committer Shawn McCarney <shawnmm@us.ibm.com> Fri Jul 17 18:37:24 2020 -0500
tree 0a43e46054549276da53cdc2b9e77087b763ca78
parent 8109502f61e630e4c86faf8771c271d77b7096c6

regulators: Modify Manager to use Services

Modify the Manager class to use the new Services interface.

Tested:
  * Verified all journal messages logged by Manager still work.

Signed-off-by: Shawn McCarney <shawnmm@us.ibm.com>
Change-Id: Ifaa1dfe67e4ab33ea42e9ee1daf65a5663087a0d

phosphor-regulators/src/manager.cpp

diff --git a/phosphor-regulators/src/manager.cpp b/phosphor-regulators/src/manager.cpp
index 10b9de9..fd8dd01 100644
--- a/phosphor-regulators/src/manager.cpp
+++ b/phosphor-regulators/src/manager.cpp
@@ -19,7 +19,6 @@
 #include "chassis.hpp"
 #include "config_file_parser.hpp"
 #include "exception_utils.hpp"
-#include "journal.hpp"
 #include "rule.hpp"
 #include "utility.hpp"
 
@@ -50,7 +49,7 @@
 const fs::path testConfigFileDir{"/etc/phosphor-regulators"};
 
 Manager::Manager(sdbusplus::bus::bus& bus, const sdeventplus::Event& event) :
-    ManagerObject{bus, objPath, true}, bus{bus}, eventLoop{event}
+    ManagerObject{bus, objPath, true}, bus{bus}, eventLoop{event}, services{bus}
 {
     /* Temporarily comment out until D-Bus interface is defined and available.
         // Subscribe to interfacesAdded signal for filename property
@@ -89,8 +88,8 @@
     }
     else
     {
-        journal::logErr("Unable to configure regulator devices: Configuration "
-                        "file not loaded");
+        services.getJournal().logError("Unable to configure regulator devices: "
+                                       "Configuration file not loaded");
         // TODO: Log error
     }
 
@@ -233,7 +232,8 @@
         fs::path pathName = findConfigFile();
 
         // Log info message in journal; config file path is important
-        journal::logInfo("Loading configuration file " + pathName.string());
+        services.getJournal().logInfo("Loading configuration file " +
+                                      pathName.string());
 
         // Parse the config file
         std::vector<std::unique_ptr<Rule>> rules{};
@@ -247,8 +247,8 @@
     catch (const std::exception& e)
     {
         // Log error messages in journal
-        exception_utils::log(e);
-        journal::logErr("Unable to load configuration file");
+        services.getJournal().logError(exception_utils::getMessages(e));
+        services.getJournal().logError("Unable to load configuration file");
 
         // TODO: Create error log entry
     }