summaryrefslogtreecommitdiff
path: root/subprojects
diff options
context:
space:
mode:
authorEd Tanous <edtanous@google.com>2022-10-06 03:21:13 +0300
committerGunnar Mills <gmills@us.ibm.com>2023-05-24 00:03:16 +0300
commit75815e5c3677236e65643944b225abf2815b69c2 (patch)
treeb74f0f17e979e07b762e28f81642d246e5932e57 /subprojects
parent479e899d5f57a67647f83b7f615d2c8565290bcf (diff)
downloadbmcweb-75815e5c3677236e65643944b225abf2815b69c2.tar.xz
Add check for "quiesced" bmc manager state
The bmc now supports the Quiesced state, which is tracked using systemd targets. Previously, the bmc startup state was determined by systemd alone. The old systemd startup behavior is retained, but if the bmc is found to be started, this commit also check the quiesced target to determine if we should set that state as well. This allows phosphor-state-manager users to have a state that works for the quiesced use case, while avoiding race conditions on startup, or having to impose a hard dependency on phosphor-state-manager, which we know some users do not use. The reasons for not using phosphor-state-manager are outside of the scope of this commit. In comparison to the alternative: https://gerrit.openbmc.org/c/openbmc/bmcweb/+/50318 This actually seems to have a smaller diff, so while there's some concern about adding complexity to bmcweb, this seems like this patch gets us the same behavior with slightly less code. Tested: Loaded onto a p10bmc and see this new state. systemctl start obmc-bmc-service-quiesce@0.target root@xxx:~# obmcutil state CurrentBMCState : xyz.openbmc_project.State.BMC.BMCState.Quiesced curl -k https://$bmc/redfish/v1/Managers/bmc ... "Status": { "Health": "Critical", "State": "Quiesced" }, Signed-off-by: Ed Tanous <edtanous@google.com> Change-Id: I718b8ad0a43327051cb5fdf0da59a1ccfbde9940 Signed-off-by: Gunnar Mills <gmills@us.ibm.com>
Diffstat (limited to 'subprojects')
0 files changed, 0 insertions, 0 deletions