Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suppress storm detect counter increment for ongoing pfc storm case during a warm reboot #869

Merged
merged 4 commits into from
Jun 4, 2019

Conversation

wendani
Copy link
Contributor

@wendani wendani commented May 1, 2019

Signed-off-by: Wenda Ni wenni@microsoft.com

What I did

Why I did it

How I verified it
Tested on brcm dut
Details if related

…ring

a warm reboot

Signed-off-by: Wenda Ni <wenni@microsoft.com>
@wendani wendani requested a review from qiluo-msft June 3, 2019 20:58
@wendani
Copy link
Contributor Author

wendani commented Jun 4, 2019

retest this please

@wendani wendani merged commit e8b8a90 into sonic-net:master Jun 4, 2019
@wendani wendani deleted the pfcwd_wb_cntr_master branch June 4, 2019 06:30
yxieca pushed a commit that referenced this pull request Jun 4, 2019
…ring a warm reboot (#869)

* Suppress storm detect counter increment for ongoing pfc storm case during
a warm reboot

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Comment touch-up

Signed-off-by: Wenda Ni <wenni@microsoft.com>
EdenGri pushed a commit to EdenGri/sonic-swss that referenced this pull request Feb 28, 2022
* Fix "First parameter of a method is not named 'self'" warnings

* Fix "Module is imported with 'import' and 'import from'" warning

* Fix 'Testing equality to None' warnings

* Clean up 'Unused import' warnings
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-swss that referenced this pull request Mar 1, 2023
…-net#869)

Start recording CRS operations response to sairedis.rec but only when response is not success.
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.

3 participants