Skip to content

Commit

Permalink
[common/reboot.py] Fix double reboot bug (#16581)
Browse files Browse the repository at this point in the history
Description of PR
Summary:
Revert double root bug introduced in #16348

Approach
What is the motivation for this PR?
Revert bug introduced in #16348 that performs 2 reboots instead of one (also performed in line 284/283)

How did you do it?
Remove one instance of reboot

Signed-off-by: Javier Tan [email protected]
  • Loading branch information
Javier-Tan authored Jan 19, 2025
1 parent aca0f87 commit a3abf26
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion tests/common/reboot.py
Original file line number Diff line number Diff line change
Expand Up @@ -275,7 +275,6 @@ def reboot(duthost, localhost, reboot_type='cold', delay=10,
duthost.command('sudo touch /dev/shm/test_reboot')
# Get reboot-cause history before reboot
prev_reboot_cause_history = duthost.show_and_parse("show reboot-cause history")
reboot_res, dut_datetime = perform_reboot(duthost, pool, reboot_command, reboot_helper, reboot_kwargs, reboot_type)

wait_conlsole_connection = 5
console_thread_res = pool.apply_async(
Expand Down

0 comments on commit a3abf26

Please sign in to comment.