From 393731cbe18a4751a6ec4d8b3858046fd7bd1ab6 Mon Sep 17 00:00:00 2001 From: Joe LeVeque Date: Sat, 27 Mar 2021 11:42:23 -0700 Subject: [PATCH] [docker-gbsyncd-vs] Run new gbsyncdmgrd in lieu of deprecated gbsyncd_startup.py (#7154) To improve management of docker-gbsyncd-vs. gbsyncd_startup.py simply spawned syncd processes and then exited. In that case, supervisord would no longer manage any processes in the container, and thus there was no way to know if a critical process had exited. I recently created gbsyncdmgrd to be a more complete, robust replacement for gbsyncd_startup.py. NOTE: This PR is dependent on the inclusion of gbsyncdmgrd in the sonic-sairedis repo. A submodule update is pending at #7089 --- platform/vs/docker-gbsyncd-vs/critical_processes | 2 +- platform/vs/docker-gbsyncd-vs/supervisord.conf | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/platform/vs/docker-gbsyncd-vs/critical_processes b/platform/vs/docker-gbsyncd-vs/critical_processes index bdd6903c5690..4775a3e8af3b 100644 --- a/platform/vs/docker-gbsyncd-vs/critical_processes +++ b/platform/vs/docker-gbsyncd-vs/critical_processes @@ -1 +1 @@ -program:syncd +program:gbsyncdmgrd diff --git a/platform/vs/docker-gbsyncd-vs/supervisord.conf b/platform/vs/docker-gbsyncd-vs/supervisord.conf index 68f2c46ab182..9b664ef1b844 100644 --- a/platform/vs/docker-gbsyncd-vs/supervisord.conf +++ b/platform/vs/docker-gbsyncd-vs/supervisord.conf @@ -37,8 +37,8 @@ stderr_logfile=syslog dependent_startup=true dependent_startup_wait_for=rsyslogd:running -[program:syncd] -command=/usr/bin/gbsyncd_startup.py +[program:gbsyncdmgrd] +command=/usr/bin/gbsyncdmgrd priority=3 autostart=false autorestart=false