commit f566c964cab791bb9427fdd31bca9f63e437de65
author Amithash Prasasd <amithash@meta.com> Mon Jul 22 20:28:42 2024 -0700
committer Amithash Prasasd <amithash@meta.com> Tue Aug 20 10:13:32 2024 -0700
tree da2c059ca235211378082d3ebfd3d740345da4c0
parent 1b2c3c035ecd647cef996982df00378506f93887

Switch to Use phosphor-dbus-interface defined constants

We are redefining a lot of the constants which are already
defined in the phosphor-dbus-interface auto-generated
headers. This change switches the code to use those instead
of allowing compile time definition of those.

Allowing for compile time definition would probably break
clients who are not aware of the new paths anyway.

Tested by build a harma image and loading in qemu to test
the added interfaces are unchanged.
```
busctl introspect xyz.openbmc_project.State.BMC \
    /xyz/openbmc_project/state/bmc0
busctl introspect xyz.openbmc_project.State.Host \
    /xyz/openbmc_project/state/host0
busctl introspect xyz.openbmc_project.State.Chassis \
    /xyz/openbmc_project/state/chassis0
```

Change-Id: Ib4c77d2789c13f509b75a2b1837ea454e53e8ae9
Signed-off-by: Amithash Prasasd <amithash@meta.com>

scheduled_host_transition.cpp

diff --git a/scheduled_host_transition.cpp b/scheduled_host_transition.cpp
index eb5975c..d2c136b 100644
--- a/scheduled_host_transition.cpp
+++ b/scheduled_host_transition.cpp
@@ -94,14 +94,18 @@
 
 void ScheduledHostTransition::hostTransition()
 {
-    auto hostPath = std::string{HOST_OBJPATH} + std::to_string(id);
+    auto hostName = std::string(HostState::namespace_path::host) +
+                    std::to_string(id);
+    std::string hostPath =
+        sdbusplus::message::object_path(HostState::namespace_path::value) /
+        hostName;
 
     auto reqTrans = convertForMessage(HostTransition::scheduledTransition());
 
     info("Trying to set requestedTransition to {REQUESTED_TRANSITION}",
          "REQUESTED_TRANSITION", reqTrans);
 
-    utils::setProperty(bus, hostPath, HOST_BUSNAME, PROPERTY_TRANSITION,
+    utils::setProperty(bus, hostPath, HostState::interface, PROPERTY_TRANSITION,
                        reqTrans);
 
     // Set RestartCause to indicate this transition is occurring due to a
@@ -111,8 +115,8 @@
         info("Set RestartCause to scheduled power on reason");
         auto resCause =
             convertForMessage(HostState::RestartCause::ScheduledPowerOn);
-        utils::setProperty(bus, hostPath, HOST_BUSNAME, PROPERTY_RESTART_CAUSE,
-                           resCause);
+        utils::setProperty(bus, hostPath, HostState::interface,
+                           PROPERTY_RESTART_CAUSE, resCause);
     }
 }