bmc-ready: auto power restart function update
This service is responsible for executing the auto power restart logic
for the system. Prior to this change, it would run as a oneshot service
during the multi-user.target. This meant the BMC would not achieve
"Ready" state until after this service has completed.
With the recent changes to support not allowing a power on of the system
until the BMC is "Ready", we need to change this service so it is not
holding up that state change.
Systems that enable this new feature to only allow a power on when the
BMC is in a "Ready" state will need to ensure they have a non-zero
PowerRestoreDelay set for their system. This allows the service in this
commit to start, the BMC to get to "Ready" and then the power on request
to be made (if needed).
If in the future there's a need to enable this new feature and also have
a 0s delay, then this service will need to be enhanced to wait for BMC
"Ready".
Tested:
- Set policy to AlwaysOn, confirmed service started, BMC hit "Ready",
service waited 30s, and then issued power on without issue.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I1ecb8ed9e1cd6babfa2c0fdb51e8a9bd6c83e214
diff --git a/service_files/phosphor-discover-system-state@.service b/service_files/phosphor-discover-system-state@.service
index 2523e5c..3f665f9 100644
--- a/service_files/phosphor-discover-system-state@.service
+++ b/service_files/phosphor-discover-system-state@.service
@@ -15,7 +15,7 @@
[Service]
Restart=no
-Type=oneshot
+Type=simple
RemainAfterExit=yes
ExecStart=/usr/bin/phosphor-discover-system-state --host %i