Skip to content

[fast/warm-reboot] Avoid the FDB clear#1116

Merged
neethajohn merged 1 commit intosonic-net:masterfrom
neethajohn:wb-no-clear-fdb
Sep 16, 2019
Merged

[fast/warm-reboot] Avoid the FDB clear#1116
neethajohn merged 1 commit intosonic-net:masterfrom
neethajohn:wb-no-clear-fdb

Conversation

@neethajohn
Copy link
Copy Markdown
Contributor

Signed-off-by: Neetha John nejo@microsoft.com

Summary:
Fixes # (issue)
t1->server packet loss was seen. Do not clear FDB entries before fast/warm reboot

Type of change

  • Bug fix
  • [] Testbed and Framework(new/improvement)
  • [] Test case(new/improvement)

How did you verify/test it?

Ran warm-reboot, fast reboot and vlan port sad path case (clear FDB was introduced was this case). No longer seeing t1->servers packet loss

Signed-off-by: Neetha John <nejo@microsoft.com>
@neethajohn neethajohn merged commit ff9e86c into sonic-net:master Sep 16, 2019
@neethajohn neethajohn deleted the wb-no-clear-fdb branch September 16, 2019 23:27
yxieca pushed a commit that referenced this pull request Sep 17, 2019
Signed-off-by: Neetha John <nejo@microsoft.com>
neethajohn referenced this pull request Oct 31, 2019
Signed-off-by: Stepan Blyschak <stepanb@mellanox.com>
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
Submodule src/sonic-swss f09ddb4..49c9c16:
  > Allow buffer profile apply after init (sonic-net#1099)
  > [aclorch]: Check for existing mirror table only when creating a new table (sonic-net#1089)
  > [201811] [portsorch] fix PortsOrch::allPortsReady() returns true when it should not (sonic-net#1116)
  > Address review comment: remove data member m_entriesCreated, which is introduced for dependancy resolution purpose. (sonic-net#839)
  > Fix PFC watchdog not getting lossless TC (sonic-net#876)

Submodule src/sonic-utilities c049e54..2ca1ae1:
  > Add a generic configlet application script (sonic-net#716)

Signed-off-by: Ying Xie <ying.xie@microsoft.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.

2 participants