Skip to content

[portsorch] fix bug in speed set#806

Merged
liat-grozovik merged 1 commit intosonic-net:masterfrom
stepanblyschak:speed_fix
Mar 6, 2019
Merged

[portsorch] fix bug in speed set#806
liat-grozovik merged 1 commit intosonic-net:masterfrom
stepanblyschak:speed_fix

Conversation

@stepanblyschak
Copy link
Copy Markdown
Contributor

Signed-off-by: Stepan Blyschak stepanb@mellanox.com

What I did
After comments handling in #764 this bug was introduced.
m_portList[alias] will be overridden with p, however p has no updated speed

Why I did it

How I verified it

Details if related

Signed-off-by: Stepan Blyschak <stepanb@mellanox.com>
@stcheng
Copy link
Copy Markdown
Contributor

stcheng commented Mar 5, 2019

@yxieca please check

@liat-grozovik liat-grozovik merged commit 3f14956 into sonic-net:master Mar 6, 2019
yxieca pushed a commit that referenced this pull request Mar 7, 2019
Signed-off-by: Stepan Blyschak <stepanb@mellanox.com>
@yxieca
Copy link
Copy Markdown
Contributor

yxieca commented Mar 7, 2019

Made to 201811 branch on 3/6/2019

oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-swss that referenced this pull request Mar 1, 2023
Bring sairedis python modules for both python2 and python3 using SWIG and extra helper methods, as well as adding debian packages for those modules.
Janetxxx pushed a commit to Janetxxx/sonic-swss that referenced this pull request Nov 10, 2025
Signed-off-by: Stepan Blyschak <stepanb@mellanox.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants