ensure proper phosphor-logging dependencies
As more errors are added to state-manager services, need to ensure they
are run after phosphor-logging is up and running. I saw an error path
recently on a BMC reset with the host up scenario where
bmc-state-manager tried to log an error but phosphor-logging was not
running yet, resulting in an unhandled exception.
Tested:
- Verify all state services start after logging (bmc is earliest):
Mar 09 18:40:33 p10bmc systemd[1]: Starting Phosphor Log Manager...
Mar 09 18:40:34 p10bmc systemd[1]: Started Phosphor Log Manager.
Mar 09 18:40:34 p10bmc systemd[1]: Starting Phosphor BMC State Manager...
Mar 09 18:40:35 p10bmc systemd[1]: Started Phosphor BMC State Manager.
- Verify PEL data after all services running:
"User Data 0": {
"Section Version": "1",
"Sub-section type": "1",
"Created by": "0x2000",
"BMCState": "Ready",
"BootState": "Unspecified",
"ChassisState": "Off",
"FW Version ID": "fw1020.00-44.2-9-g1186c2a5f8",
"HostState": "Off",
"System IM": "50001001"
},
- Verify PEL data when BMC state service is stopped, and logging service
is restarted (to mimic error prior to state service running)
"User Data 0": {
"Section Version": "1",
"Sub-section type": "1",
"Created by": "0x2000",
"BMCState": "",
"BootState": "Unspecified",
"ChassisState": "Off",
"FW Version ID": "fw1020.00-44.2-9-g1186c2a5f8",
"HostState": "Off",
"System IM": "50001001"
},
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: Id9528ab911fc57b400e40fdfc9353f4432f57d63
diff --git a/service_files/xyz.openbmc_project.State.ScheduledHostTransition.service b/service_files/xyz.openbmc_project.State.ScheduledHostTransition.service
index 3d08060..9964724 100644
--- a/service_files/xyz.openbmc_project.State.ScheduledHostTransition.service
+++ b/service_files/xyz.openbmc_project.State.ScheduledHostTransition.service
@@ -5,6 +5,8 @@
After=obmc-mapper.target
Wants=xyz.openbmc_project.State.Host.service
After=xyz.openbmc_project.State.Host.service
+Wants=xyz.openbmc_project.Logging.service
+After=xyz.openbmc_project.Logging.service
[Service]
ExecStart=/usr/bin/phosphor-scheduled-host-transition
@@ -13,4 +15,4 @@
BusName=xyz.openbmc_project.State.ScheduledHostTransition
[Install]
-WantedBy=multi-user.target
\ No newline at end of file
+WantedBy=multi-user.target