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

[VxLAN]Fix Vxlan delete command to throw error when there are references #2404

Merged
merged 1 commit into from
Sep 29, 2022

Conversation

dgsudharsan
Copy link
Collaborator

@dgsudharsan dgsudharsan commented Sep 27, 2022

Fix for sonic-net/sonic-buildimage#12063

What I did

Check for VNET references for vxlan before deleting vxlan object

How I did it

Check VNET table and throw error if vxlan is referenced

How to verify it

Added UT 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)

@prsunny prsunny merged commit ac71d74 into sonic-net:master Sep 29, 2022
yxieca pushed a commit that referenced this pull request Oct 3, 2022
…ces (#2404)

*Check for VNET references for vxlan before deleting vxlan object
qiluo-msft pushed a commit that referenced this pull request Oct 3, 2022
…ces (#2404)

*Check for VNET references for vxlan before deleting vxlan object
dprital added a commit to dprital/sonic-buildimage that referenced this pull request Oct 11, 2022
Update sonic-utilities submodule pointer to include the following:
* 94a3436 [show] vnet endpoint [ip/ipv6] command (sonic-net#2342) ([sonic-net#2421](sonic-net/sonic-utilities#2421))
* 84a0712 [VxLAN]Fix Vxlan delete command to throw error when there are references ([sonic-net#2404](sonic-net/sonic-utilities#2404))
* 1341f58 [202012] [generate_dump]: Enhance show techsupport for cisco-8000 platform ([sonic-net#2395](sonic-net/sonic-utilities#2395))

Signed-off-by: dprital <drorp@nvidia.com>
EdenGri pushed a commit to EdenGri/sonic-utilities that referenced this pull request Oct 12, 2022
…ces (sonic-net#2404)

*Check for VNET references for vxlan before deleting vxlan object
mdanish-kh pushed a commit to hamnarauf/sonic-utilities that referenced this pull request Oct 22, 2022
…ces (sonic-net#2404)

*Check for VNET references for vxlan before deleting vxlan object
preetham-singh pushed a commit to preetham-singh/sonic-utilities that referenced this pull request Nov 21, 2022
…ces (sonic-net#2404)

*Check for VNET references for vxlan before deleting vxlan object
@dgsudharsan dgsudharsan deleted the vxlan_vnet_check branch March 9, 2023 02:04
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.

6 participants