[teamd]: Downgrade supervisord (python3->python2) to overcome LAG cleanup degradation #24
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…anup degradation. Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
Comment on lines
+26
to
+34
| # W/A: setuptools and wheel are necessary for installing some Python wheel packages | ||
| RUN pip2 install --no-cache-dir setuptools==44.1.1 | ||
| RUN pip2 install --no-cache-dir wheel==0.35.1 | ||
|
|
||
| # W/A: install supervisor | ||
| RUN pip2 install supervisor==4.2.1 | ||
|
|
||
| # W/A: add support for supervisord to handle startup dependencies | ||
| RUN pip2 install supervisord-dependent-startup==1.4.0 |
There was a problem hiding this comment.
Can we combine into single RUN instruction to reduce build time?
There was a problem hiding this comment.
@stepanblyschak this pattern has been taken from the original Makefile and it has some logical blocks separation. Not sure if we can gain much performance from combining install steps.
There was a problem hiding this comment.
It does make a difference: https://github.com/Azure/SONiC/blob/master/doc/sonic-build-system/build_system_improvements.md#improving-dockerfile-instructions.
We do not gain anything from seperate RUN instructions since cache is disabled.
stepanblyschak
approved these changes
Jul 21, 2021
nazariig
pushed a commit
that referenced
this pull request
Mar 30, 2022
ce72b0d Longxiang Lyu Thu Feb 24 06:05:12 2022 Put handler member functions as virtual in base (#30) ef59e4f Jing Zhang Fri Feb 25 11:38:28 2022 Incrementing tolerance on mux state inconsistency (#27) 2d12892 Longxiang Lyu Wed Feb 16 03:32:06 2022 Rename LinkManagerStateMachine to ActiveStandbyStateMachine (#26) f38634c Jing Zhang Thu Feb 17 17:23:56 2022 Update log level for mux probing and mux state chance (#23) a8434dd Jing Zhang Thu Feb 17 17:21:01 2022 Handle xcvrd crashing scenarios (#22) 2ebdb2b Longxiang Lyu Mon Feb 14 13:26:07 2022 [make] Enable make extra includes (#24)
nazariig
pushed a commit
that referenced
this pull request
Jun 15, 2022
Changes: Update submodule branch to 202012 [sonic-linkmgrd][202012] submodule update a8ddff5 Jing Zhang Fri Feb 25 11:38:28 2022 Incrementing tolerance on mux state inconsistency (#27) a3f78a3 Jing Zhang Thu Feb 17 17:23:56 2022 Update log level for mux probing and mux state chance (#23) 05156fb Jing Zhang Thu Feb 17 17:21:01 2022 Handle xcvrd crashing scenarios (#22) 74529ef Longxiang Lyu Mon Feb 14 13:26:07 2022 [make] Enable make extra includes (#24) sign-off: Jing Zhang zhangjing@microsoft.com
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Nazarii Hnydyn nazariig@nvidia.com
Why I did it
How I did it
How to verify it
Which release branch to backport (provide reason below if selected)
Description for the changelog
A picture of a cute animal (not mandatory but encouraged)