diff --git a/src/sonic-host-services/scripts/determine-reboot-cause b/src/sonic-host-services/scripts/determine-reboot-cause index 8439c78a7016..8f1ee0a39b5d 100755 --- a/src/sonic-host-services/scripts/determine-reboot-cause +++ b/src/sonic-host-services/scripts/determine-reboot-cause @@ -164,35 +164,22 @@ def main(): if os.path.exists(PREVIOUS_REBOOT_CAUSE_FILE): os.remove(PREVIOUS_REBOOT_CAUSE_FILE) - hardware_reboot_cause = None # This variable is kept for future-use purpose. When proc_cmd_line/vendor/software provides # any additional_reboot_info it will be stored as a "comment" in REBOOT_CAUSE_HISTORY_FILE additional_reboot_info = "N/A" - # 1. Check if the previous reboot was warm/fast reboot by testing whether there is "fast|fastfast|warm" in /proc/cmdline + # Check if the previous reboot was warm/fast reboot by testing whether there is "fast|fastfast|warm" in /proc/cmdline proc_cmdline_reboot_cause = find_proc_cmdline_reboot_cause() - # 2. Check if the previous reboot was caused by hardware - # If yes, the hardware reboot cause will be treated as the reboot cause - hardware_reboot_cause = find_hardware_reboot_cause() - - # 3. If there is a REBOOT_CAUSE_FILE, it will contain any software-related - # reboot info. We will use it as the previous cause. - software_reboot_cause = find_software_reboot_cause() - - # The main decision logic of the reboot cause: - # If there is a reboot cause indicated by /proc/cmdline, it should be warmreboot/fastreboot - # the software_reboot_cause which is the content of /hosts/reboot-cause/reboot-cause.txt - # will be treated as the reboot cause - # Elif there is a reboot cause indicated by platform API, - # the hardware_reboot_cause will be treated as the reboot cause - # Else the software_reboot_cause will be treated as the reboot cause - if proc_cmdline_reboot_cause is not None: - previous_reboot_cause = software_reboot_cause - elif hardware_reboot_cause is not REBOOT_CAUSE_NON_HARDWARE: - previous_reboot_cause = hardware_reboot_cause + # If /proc/cmdline does not indicate reboot cause, check if the previous reboot was caused by hardware + if proc_cmdline_reboot_cause is None: + previous_reboot_cause = find_hardware_reboot_cause() + if previous_reboot_cause.startswith(REBOOT_CAUSE_NON_HARDWARE): + # If the reboot cause is non-hardware, get the reboot cause from REBOOT_CAUSE_FILE + previous_reboot_cause = find_software_reboot_cause() else: - previous_reboot_cause = software_reboot_cause + # Get the reboot cause from REBOOT_CAUSE_FILE + previous_reboot_cause = find_software_reboot_cause() # Current time reboot_cause_gen_time = str(datetime.datetime.now().strftime('%Y_%m_%d_%H_%M_%S'))