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

Revert "[vlanmgr] Disable arp_evict_nocarrier for vlan host intf (#… #2518

Merged
merged 1 commit into from
Nov 14, 2022

Conversation

lolyu
Copy link
Contributor

@lolyu lolyu commented Nov 14, 2022

This reverts commit 7290d66.

What I did

Why I did it

How I verified it

Details if related

@lolyu lolyu requested a review from prsunny as a code owner November 14, 2022 03:17
@liuh-80 liuh-80 requested review from liuh-80 and qiluo-msft and removed request for liuh-80 November 14, 2022 03:24
@liuh-80
Copy link
Contributor

liuh-80 commented Nov 14, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80 liuh-80 merged commit 4a165ee into sonic-net:202205 Nov 14, 2022
@prsunny
Copy link
Collaborator

prsunny commented Nov 14, 2022

@lolyu , should this be reverted from any other branch, say 202012?

lolyu added a commit that referenced this pull request Jan 4, 2023
What I did
Cherry-pick back #2469 into 202205.

Why I did it
It is reverted in 202205: #2518 because of the flakiness of test_vlan.py

How I verified it
This depends on the following PRs:
#2504
#2541
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants