From 2aa1c2f4374cf3db2d8019f218d505d5521949f1 Mon Sep 17 00:00:00 2001 From: Stephen Sun <5379172+stephenxs@users.noreply.github.com> Date: Sat, 21 May 2022 15:13:49 +0800 Subject: [PATCH] Fix issue: error message from system-health daemon is observed during system starting (#10843) - Why I did it Error message: "ERR healthd: Failed to read from file /var/run/hw-management/led/led_status_capability" is observed during system starting The system-health daemon will wait for 5 minutes before it starts to run. During this time, the only thing it does is to set the LED even before it starts. However, the corresponding sysfs has not been ready at the time it is being read, which causes the error message. - How I did it Defer system-health daemon until hw-management service starts - How to verify it Run regression test Signed-off-by: Stephen Sun --- ...m-health-service-service-start-after.patch | 27 +++++++++++++++++++ 1 file changed, 27 insertions(+) create mode 100644 platform/mellanox/hw-management/0004-Make-SONiC-system-health-service-service-start-after.patch diff --git a/platform/mellanox/hw-management/0004-Make-SONiC-system-health-service-service-start-after.patch b/platform/mellanox/hw-management/0004-Make-SONiC-system-health-service-service-start-after.patch new file mode 100644 index 0000000000..95bf7ae919 --- /dev/null +++ b/platform/mellanox/hw-management/0004-Make-SONiC-system-health-service-service-start-after.patch @@ -0,0 +1,27 @@ +From 75725ff405c65a6aba0b936bc2bee6f04aaaed74 Mon Sep 17 00:00:00 2001 +From: Stephen Sun +Date: Thu, 12 May 2022 08:41:00 +0800 +Subject: [PATCH] Make SONiC system-health service service start after hw-mgmt + service + +Signed-off-by: Stephen Sun +--- + debian/hw-management.hw-management.service | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +diff --git a/debian/hw-management.hw-management.service b/debian/hw-management.hw-management.service +index 1c25ffb..9d10a32 100755 +--- a/debian/hw-management.hw-management.service ++++ b/debian/hw-management.hw-management.service +@@ -1,7 +1,7 @@ + [Unit] + Description=Chassis HW management service of Mellanox systems + Documentation=man:hw-management.service(8) +-Before=determine-reboot-cause.service ++Before=determine-reboot-cause.service system-health.service + + [Service] + Type=oneshot +-- +1.9.1 +