commit 8feb04544ae69154a47c4323f5ada2e6da34f50e
author Alexander Hansen <alexander.hansen@9elements.com> Mon Sep 15 14:29:20 2025 +0200
committer Alexander Hansen <alexander.hansen@9elements.com> Mon Oct 13 14:44:19 2025 +0200
tree d4d607002fd752a96bf9f95b42e7ba43310e0b08
parent a915884a6d909fc2bf12daa85cd9424f7df6320f

use phosphor-logging

Migrate all instances of std::cerr and std::cout to phosphor-logging
such as lg2::error, lg2::info, lg2::debug, ...

The use of modern logging infrastructure helps with development since
additional logging levels such as `debug` and `warning` are available.

Migrating the remaining logging calls helps to make the code more
readable and uniform.

Tested: Inspection, and on Tyan S8030

Restarted EM with patch applied.
Logs appear as usual.

```
Sep 15 13:52:46 s8030-bmc-30303035c0c1 entity-manager[23480]: Inventory Added: Supermicro PWS 920P SQ 0
Sep 15 13:52:46 s8030-bmc-30303035c0c1 entity-manager[23480]: Inventory Added: Supermicro PWS 920P SQ 1
Sep 15 13:52:46 s8030-bmc-30303035c0c1 entity-manager[23480]: Inventory Added: Tyan S8030 Baseboard
Sep 15 13:52:46 s8030-bmc-30303035c0c1 entity-manager[23480]: Inventory Added: MBX 1.57 Chassis
```

`busctl tree` output appears as before.

Did a power cycle to trigger a few more log prints

```
Sep 15 13:55:14 s8030-bmc-30303035c0c1 entity-manager[23480]: power match triggered
```

Created configuration file with missing fields to trigger error print.
```
{
  "Exposes": [],
  "Type": "error"
}
```

```
Sep 15 13:56:58 s8030-bmc-30303035c0c1 entity-manager[23659]: Finished loading json configuration in 7938ms
Sep 15 13:56:58 s8030-bmc-30303035c0c1 entity-manager[23659]: configuration file missing probe:
                                                               {"Exposes":[],"Type":"error"}
Sep 15 13:56:59 s8030-bmc-30303035c0c1 entity-manager[23659]: configuration file missing probe:
                                                               {"Exposes":[],"Type":"error"}
```

Change-Id: I3452f983c9c14cd02ab9b56451c4b3e4a13c3979
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

src/entity_manager/dbus_interface.hpp

diff --git a/src/entity_manager/dbus_interface.hpp b/src/entity_manager/dbus_interface.hpp
index 9bf223f..9c20d8f 100644
--- a/src/entity_manager/dbus_interface.hpp
+++ b/src/entity_manager/dbus_interface.hpp
@@ -4,10 +4,10 @@
 
 #include <boost/container/flat_map.hpp>
 #include <nlohmann/json.hpp>
+#include <phosphor-logging/lg2.hpp>
 #include <sdbusplus/asio/connection.hpp>
 #include <sdbusplus/asio/object_server.hpp>
 
-#include <iostream>
 #include <vector>
 
 namespace dbus_interface
@@ -89,12 +89,12 @@
                 if (!setJsonFromPointer(jsonPointerString, val,
                                         systemConfiguration))
                 {
-                    std::cerr << "error setting json field\n";
+                    lg2::error("error setting json field");
                     return -1;
                 }
                 if (!writeJsonFiles(systemConfiguration))
                 {
-                    std::cerr << "error setting json file\n";
+                    lg2::error("error setting json file");
                     return -1;
                 }
                 return 1;
@@ -123,12 +123,12 @@
             if (!setJsonFromPointer(jsonPointerString, val,
                                     systemConfiguration))
             {
-                std::cerr << "error setting json field\n";
+                lg2::error("error setting json field");
                 return -1;
             }
             if (!writeJsonFiles(systemConfiguration))
             {
-                std::cerr << "error setting json file\n";
+                lg2::error("error setting json file");
                 return -1;
             }
             return 1;