Skip to content

Commit

Permalink
[docker-gbsyncd-vs] Run new gbsyncdmgrd in lieu of deprecated gbsyncd…
Browse files Browse the repository at this point in the history
…_startup.py (sonic-net#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
sonic-net#7089
  • Loading branch information
jleveque authored and Carl Keene committed Aug 7, 2021
1 parent 293360a commit 393731c
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion platform/vs/docker-gbsyncd-vs/critical_processes
Original file line number Diff line number Diff line change
@@ -1 +1 @@
program:syncd
program:gbsyncdmgrd
4 changes: 2 additions & 2 deletions platform/vs/docker-gbsyncd-vs/supervisord.conf
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 393731c

Please sign in to comment.