Skip to content

Commit

Permalink
Fix syncd_request_shutdown coredump in config reload on KVM sonic (so…
Browse files Browse the repository at this point in the history
…nic-net#17486)

The issue is related to sonic-net#16812. Process syncd does not run in the container gbsyncd on kvm sonic with default hwsku.

Microsoft ADO : 26151608

How I did it
If syncd has not run in container gbsyncd, it is not needed to trigger graceful shudown of syncd.

How to verify it
None of syncd_request_shutdown coredump in config reload on KVM sonic
  • Loading branch information
jimmyzhai authored and mssonicbld committed Dec 19, 2023
1 parent d7ba060 commit 80e8781
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions files/scripts/gbsyncd.sh
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,11 @@ function waitplatform() {
}

function stopplatform1() {
if ! docker top gbsyncd$DEV | grep -q /usr/bin/syncd; then
debug "syncd process in container gbsyncd$DEV is not running"
return
fi

# Invoke platform specific pre shutdown routine.
PLATFORM=`$SONIC_DB_CLI CONFIG_DB hget 'DEVICE_METADATA|localhost' platform`
PLATFORM_PRE_SHUTDOWN="/usr/share/sonic/device/$PLATFORM/plugins/gbsyncd_request_pre_shutdown"
Expand Down

0 comments on commit 80e8781

Please sign in to comment.