Joe LeVeque
dd9be59cd1
[202012][dockers][supervisor] Increase event buffer size for process exit listener; Set all event buffer sizes to 1024 ( #7203 )
...
#### Why I did it
Backport of https://github.com/Azure/sonic-buildimage/pull/7083 to the 202012 branch.
To prevent error [messages](https://dev.azure.com/mssonic/build/_build/results?buildId=2254&view=logs&j=9a13fbcd-e92d-583c-2f89-d81f90cac1fd&t=739db6ba-1b35-5485-5697-de102068d650&l=802 ) like the following from being logged:
```
Mar 17 02:33:48.523153 vlab-01 INFO swss#supervisord 2021-03-17 02:33:48,518 ERRO pool supervisor-proc-exit-listener event buffer overflowed, discarding event 46
```
This is basically an addendum to https://github.com/Azure/sonic-buildimage/pull/5247 , which increased the event buffer size for dependent-startup. While supervisor-proc-exit-listener doesn't subscribe to as many events as dependent-startup, there is still a chance some containers (like swss, as in the example above) have enough processes running to cause an overflow of the default buffer size of 10.
This is especially important for preventing erroneous log_analyzer failures in the sonic-mgmt repo regression tests, which have started occasionally causing PR check builds to fail. Example [here](https://dev.azure.com/mssonic/build/_build/results?buildId=2254&view=logs&j=9a13fbcd-e92d-583c-2f89-d81f90cac1fd&t=739db6ba-1b35-5485-5697-de102068d650&l=802 ).
I set all supervisor-proc-exit-listener event buffer sizes to 1024, and also updated all dependent-startup event buffer sizes to 1024, as well, to keep things simple, unified, and allow headroom so that we will not need to adjust these values frequently, if at all.
2021-04-01 12:52:19 -07:00
Joe LeVeque
7bf05f7f4f
[supervisor] Install vanilla package once again, install Python 3 version in Buster container ( #5546 )
...
**- Why I did it**
We were building a custom version of Supervisor because I had added patches to prevent hangs and crashes if the system clock ever rolled backward. Those changes were merged into the upstream Supervisor repo as of version 3.4.0 (http://supervisord.org/changes.html#id9 ), therefore, we should be able to simply install the vanilla package via pip. This will also allow us to easily move to Python 3, as Python 3 support was added in version 4.0.0.
**- How I did it**
- Remove Makefiles and patches for building supervisor package from source
- Install Python 3 supervisor package version 4.2.1 in Buster base container
- Also install Python 3 version of supervisord-dependent-startup in Buster base container
- Debian package installed binary in `/usr/bin/`, but pip package installs in `/usr/local/bin/`, so rather than update all absolute paths, I changed all references to simply call `supervisord` and let the system PATH find the executable to prevent future need for changes just in case we ever need to switch back to build a Debian package, then we won't need to modify these again.
- Install Python 2 supervisor package >= 3.4.0 in Stretch and Jessie base containers
2020-11-19 23:41:32 -08:00
Guohan Lu
267b0b7aa8
[docker-iccpd]: use service dependency in supervisord to start services
...
Signed-off-by: Guohan Lu <lguohan@gmail.com>
2020-05-22 11:01:28 -07:00
shine4chen
524cf9e56a
MCLAG feature for SONIC ( #2514 )
...
* MCLAG feature for sonic
* MCLAG feature for sonic
* remove binary file
* remove unused dockerfile
update docker-iccpd to stretch-based container
Signed-off-by: shine.chen <shine.chen@nephosinc.com>
* minor fix for isolation port setting
Signed-off-by: shine.chen <shine.chen@nephosinc.com>
* iccpd docker would start on demand
Signed-off-by: shine.chen <shine.chen@nephosinc.com>
* Add x attribute on mclagdctl file
Signed-off-by: shine.chen <shine.chen@nephosinc.com>
* add warm-reboot support for MCLAG
Signed-off-by: shine.chen <shine.chen@nephosinc.com>
* merge to master branch and reformat iccpd file
Signed-off-by: shine.chen <shine.chen@nephosinc.com>
* fix some bugs and make peer-link configuration optional
Signed-off-by: shine.chen <shine.chen@nephosinc.com>
* refactor code per Brcm review
Signed-off-by: shine.chen <shine.chen@nephosinc.com>
* correct a typo
Signed-off-by: shine.chen <shine.chen@nephosinc.com>
* * optimize iccpd arp/mac sync process
* refine code according to brcm opinoin
* unify function return value
Signed-off-by: shine.chen <shine.chen@nephosinc.com>
* * optimize warm-reboot process
* estabish iccpd connection with configurated src-ip
Signed-off-by: shine.chen <shine.chen@nephosinc.com>
* fix a typo
Signed-off-by: shine.chen <shine.chen@nephosinc.com>
* optimize some code
* add some debug info
* optimize bridge mac setting
* fix vlan mac sync issue on standby node
Signed-off-by: shine.chen <shine.chen@mediatek.com>
* optimize some code
Signed-off-by: shine.chen <shine.chen@mediatek.com>
* fix some bugs for warm-reboot
Signed-off-by: shine.chen <shine.chen@mediatek.com>
* refine log level
Signed-off-by: shine.chen <shine.chen@mediatek.com>
* refine iccpd syslog & skip arp packet whose src ip is local ip
Signed-off-by: shine.chen <shine.chen@mediatek.com>
* remove iccpd dependency with teamd
Signed-off-by: shine.chen <shine.chen@mediatek.com>
* print log level when dump mclag status
Signed-off-by: shine.chen <shine.chen@mediatek.com>
* revise per community review
Signed-off-by: shine.chen <shine.chen@mediatek.com>
Co-authored-by: shine.chen <shine.chen@nephosinc.com>
Co-authored-by: shine.chen <shine.chen@mediatek.com>
2020-04-04 15:24:06 -07:00
