Skip to content

Revert "[config reload]: On dual ToR systems, cache ARP and FDB table…#2460

Merged
theasianpianist merged 2 commits intosonic-net:masterfrom
theasianpianist:revert-arp-cache
Oct 28, 2022
Merged

Revert "[config reload]: On dual ToR systems, cache ARP and FDB table…#2460
theasianpianist merged 2 commits intosonic-net:masterfrom
theasianpianist:revert-arp-cache

Conversation

@theasianpianist
Copy link
Contributor

…s (#1465)"

This reverts commit 10de91d.

Signed-off-by: Lawrence Lee lawlee@microsoft.com

What I did

How I did it

How to verify it

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

sonic-net#1465)"

This reverts commit 10de91d.

Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
yxieca
yxieca previously approved these changes Oct 27, 2022
Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
@qiluo-msft
Copy link
Contributor

This commit could not be cleanly cherry-picked to 202012. Please submit another PR.

theasianpianist added a commit to theasianpianist/sonic-utilities that referenced this pull request Nov 10, 2022
sonic-net#2460)

…s (sonic-net#1465)"

- This reverts commit 10de91d.
- Also removes '--disable-arp-cache' option from config reload tests that was added in sonic-net#2325

Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
@theasianpianist
Copy link
Contributor Author

This commit could not be cleanly cherry-picked to 202012. Please submit another PR.

202012 PR here, please check #2490

preetham-singh pushed a commit to preetham-singh/sonic-utilities that referenced this pull request Nov 21, 2022
sonic-net#2460)

…s (sonic-net#1465)"

- This reverts commit 10de91d.
- Also removes '--disable-arp-cache' option from config reload tests that was added in sonic-net#2325 

Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
@liuh-80
Copy link
Contributor

liuh-80 commented Feb 2, 2023

202012 branch PR merged: #2490

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