commit f2e53726607d66713af503541e869565b698b912
author Andrew Geissler <geissonator@yahoo.com> Tue May 03 11:18:06 2022 -0400
committer Andrew Geissler <geissonator@yahoo.com> Wed May 04 09:29:35 2022 -0400
tree 8834881962e4aeb6965d91dd49d9fc4ac633b667
parent 67513c143748b7f773542c7e79773222895dbbfd

pvm_sys_dump_active: ensure no failure

Setting this BIOS attribute is a "best try" type function. If it fails,
do not cause this service to fail. The BIOS is used to provide a warning
to the user but is not a hard requirement.

Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I43a12c2bb54dd6093195bec5bdfd760997981a94

service_files/op-enter-mpreboot@.service.in

diff --git a/service_files/op-enter-mpreboot@.service.in b/service_files/op-enter-mpreboot@.service.in
index a130b9c..5407b5a 100644
--- a/service_files/op-enter-mpreboot@.service.in
+++ b/service_files/op-enter-mpreboot@.service.in
@@ -19,7 +19,8 @@
   "busctl set-property xyz.openbmc_project.BIOSConfigManager \
    /xyz/openbmc_project/bios_config/manager xyz.openbmc_project.BIOSConfig.Manager \
    PendingAttributes a{s\\(sv\\)} 1 pvm_sys_dump_active \
-   xyz.openbmc_project.BIOSConfig.Manager.AttributeType.Enumeration s Enabled"
+   xyz.openbmc_project.BIOSConfig.Manager.AttributeType.Enumeration s Enabled" \
+   || true
 ExecStart=/bin/sh -c \
   "busctl call  xyz.openbmc_project.Dump.Manager /xyz/openbmc_project/dump/bmc \
    xyz.openbmc_project.Dump.Create CreateDump a{sv} 0"  || true