Skip to content

Undo skipping Vnet tests, Vrf check before enslaving#857

Merged
prsunny merged 3 commits intosonic-net:masterfrom
prsunny:vnet
May 8, 2019
Merged

Undo skipping Vnet tests, Vrf check before enslaving#857
prsunny merged 3 commits intosonic-net:masterfrom
prsunny:vnet

Conversation

@prsunny
Copy link
Collaborator

@prsunny prsunny commented Apr 25, 2019

What I did
Re-enable Vnet Vxlan tests.
Check VRF name before setting it as master. It is possible to have INTERFACE table without the vrfname for the default VRF case. Check added to handle this.

Why I did it
Dependency with sonic-net/sonic-buildimage#2705 resolved

How I verified it
N/A

Details if related
N/A

@prsunny
Copy link
Collaborator Author

prsunny commented Apr 26, 2019

retest this please

@prsunny prsunny changed the title Undo skipping Vnet tests, VxlanMgrd merged to master Undo skipping Vnet tests, Vrf check before enslaving Apr 30, 2019
@prsunny
Copy link
Collaborator Author

prsunny commented May 7, 2019

retest this please

@prsunny prsunny merged commit 015fc62 into sonic-net:master May 8, 2019
@prsunny prsunny deleted the vnet branch February 7, 2020 22:56
EdenGri pushed a commit to EdenGri/sonic-swss that referenced this pull request Feb 28, 2022
Changes:
1.) Implement a class, which uses hsetnx for lock.
2.) lock is expired within timeout period or will be released by owner.
3.) After -y prompt, lock is reacquired, because timer could have expired,
    before user enters yes.
Signed-off-by: Praveen Chaudhary pchaudhary@linkedin.com
EdenGri pushed a commit to EdenGri/sonic-swss that referenced this pull request Feb 28, 2022
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-swss that referenced this pull request Mar 1, 2023
Janetxxx pushed a commit to Janetxxx/sonic-swss that referenced this pull request Nov 10, 2025
* Undo skipping Vnet tests, VxlanMgrd merged to master

* Check VRF name before setting as master
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.

2 participants