Modify debug message for multi-host environment
Description:
Add an ID number in the message to indicate which Host/Chassis
is executing the state transition.
Motivation:
In multi-host machines, the journal logs do not clearly show which
host is executing.
Test Log:
'''
Aug 28 06:51:41 bmc phosphor-chassis-state-manager[768]: Change to Chassis1 Power State: xyz.openbmc_project.State.Chassis.PowerState.On
Aug 28 06:51:49 bmc phosphor-discover-system-state[2166]: Host2 power is off, processing power policy xyz.openbmc_project.Control.Power.RestorePolicy.Policy.AlwaysOn
Aug 28 06:51:49 bmc phosphor-discover-system-state[2166]: power_policy=ALWAYS_POWER_ON, powering host2 on (300s delay)
Aug 28 06:51:49 bmc phosphor-host-state-manager[1305]: Host1 state transition request of xyz.openbmc_project.State.Host.Transition.On
Aug 28 06:52:00 bmc phosphor-host-state-manager[1305]: Change to Host1 State: xyz.openbmc_project.State.Host.HostState.Running
Aug 28 07:04:13 bmc phosphor-chassis-state-manager[768]: Change to Chassis1 Requested Power State: xyz.openbmc_project.State.Chassis.Transition.Off
'''
Change-Id: I4db0e7bd361346e4d659b30b9c00587a35292f3a
Signed-off-by: Zoey YJ Chung <zoey.yj.chung.wiwynn@gmail.com>
diff --git a/host_state_manager.cpp b/host_state_manager.cpp
index fbb2123..ac9181d 100644
--- a/host_state_manager.cpp
+++ b/host_state_manager.cpp
@@ -406,7 +406,8 @@
Host::Transition Host::requestedHostTransition(Transition value)
{
- info("Host state transition request of {REQ}", "REQ", value);
+ info("Host{HOST_ID} state transition request of {REQ}", "HOST_ID", id,
+ "REQ", value);
#if ONLY_ALLOW_BOOT_WHEN_BMC_READY
if ((value != Transition::Off) && (!utils::isBmcReady(this->bus)))
@@ -462,7 +463,8 @@
Host::HostState Host::currentHostState(HostState value)
{
- info("Change to Host State: {STATE}", "STATE", value);
+ info("Change to Host{HOST_ID} State: {STATE}", "HOST_ID", id, "STATE",
+ value);
return server::Host::currentHostState(value);
}