Skip to content

Commit

Permalink
[fast reboot] stop removing opennsl module before reboot (sonic-net#560)
Browse files Browse the repository at this point in the history
We have been running warm reboot test without removing opennsl for
10k+ iterations and not having issue.

The kexec reboot part is the same for warm reboot and fast reboot. It
should be safe for use to skip this during fast reboot to save some
time too.

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
  • Loading branch information
yxieca authored Jul 10, 2019
1 parent 37823d4 commit 1aac696
Showing 1 changed file with 0 additions and 9 deletions.
9 changes: 0 additions & 9 deletions scripts/fast-reboot
Original file line number Diff line number Diff line change
Expand Up @@ -430,15 +430,6 @@ docker ps -q | xargs docker kill > /dev/null
# Stop the docker container engine. Otherwise we will have a broken docker storage
systemctl stop docker.service
if [[ "$REBOOT_TYPE" != "warm-reboot" ]]; then
# Stop opennsl modules for Broadcom platform
if [[ "$sonic_asic_type" = 'broadcom' ]];
then
service_name=$(systemctl list-units --plain --no-pager --no-legend --type=service | grep opennsl | cut -f 1 -d' ')
systemctl stop "$service_name"
fi
fi
# Stop kernel modules for Nephos platform
if [[ "$sonic_asic_type" = 'nephos' ]];
then
Expand Down

0 comments on commit 1aac696

Please sign in to comment.