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

[Yang Models] Add Yang Model and tests for VNET_ROUTE_TUNNEL table #21814

Closed
wants to merge 52 commits into from

Conversation

miatttao
Copy link
Contributor

@miatttao miatttao commented Feb 20, 2025

Why I did it

Added YANG model and tests for VNET_ROUTE_TUNNEL table to provide schema validation and testing capabilities for VNET route tunnel configurations.

Work item tracking
  • Microsoft ADO (number only):

How I did it

  • Created new YANG model (sonic-vnet-route-tunnel-vxlan.yang) defining the VNET_ROUTE_TUNNEL table schema
  • Added comprehensive test cases covering both valid and invalid configurations
  • Implemented validation for:
    • Mandatory endpoint field
    • MAC address format
    • VXLAN ID range
    • Name format (vnet_name:prefix)
    • Unique reference integrity for VXLAN configurations

How to verify it

  1. Review the YANG model against the config_db schema
  2. cd src/sonic-yang-models && python3 setup.py test

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

https://github.com/Azure/sonic-buildimage/blob/master/src/sonic-yang-models/doc/Configuration.md/VNET_ROUTE_TUNNEL

A picture of a cute animal (not mandatory but encouraged)

@miatttao miatttao requested a review from qiluo-msft as a code owner February 20, 2025 22:02
Copy link

linux-foundation-easycla bot commented Feb 20, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@miatttao miatttao changed the title [Yang Models] Add Young Model and tests for VNET_ROUTE_TUNNEL table [Yang Models] Add YangModel and tests for VNET_ROUTE_TUNNEL table Feb 21, 2025
@miatttao miatttao changed the title [Yang Models] Add YangModel and tests for VNET_ROUTE_TUNNEL table [Yang Models] Add Yang Model and tests for VNET_ROUTE_TUNNEL table Feb 21, 2025
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

tjchadaga and others added 22 commits March 5, 2025 17:21
…sonic-net#21793)

Why I did it
To update the Broadcom DNX SAI version to version 12.3 and update the kernel modules

Work item tracking
Microsoft ADO (number only): 31039883

How I did it
Build 12.3 SAI debian and update SAI and kernel modules accordingly

How to verify it
Build and run basic sonic-mgmt tests on DNX platform
…in STATE_DB (sonic-net#21813)

Currently Management interface related data is present in CONFIG_DB and 'oper_status' is present in STATE_DB MGMT_PORT_TABLE.
This PR change is made to ensure that all telemetry data related to management interface is present in STATE_DB.

Signed-off-by: Suvarna Meenakshi <sumeenak@microsoft.com>
Remove PG 3 and 4 for Internal Ports
…et#19652)

[docker-orchagent] add -R flag for ring mode in orchagent.sh
Why I did it
In a subsequent PR in the sonic-mgmt repo, I will add RADIUS tests, and there needs to be a consumable RADIUS server in the infrastructure.

How I did it
Added the freeradius package the ptf docker j2 file.

How to verify it
I build the new docker image locally and then ran few succesfull app-topo tasks inside of sonic-mgmt. Also verifed that freeradius is installed.
…atically (sonic-net#21343)

#### Why I did it
src/sonic-utilities
```
* 899ed9b9 - (HEAD -> master, origin/master, origin/HEAD) Remove Multi ASIC namespace Check. (sonic-net#3783) (3 days ago) [Xincun Li]
* 9f1eab43 - Memory Statistics Config and Show Commands (sonic-net#3575) (3 days ago) [Kanza Latif]
* 595c2aaa - Utilities Changes for DHCP DoS Mitigation Feature (sonic-net#3301) (3 days ago) [Asad Raza]
* a3d15bc1 - Add multi-asic support for dropconfig (sonic-net#3735) (3 days ago) [HP]
* 5ce06b27 - Add golden config check (sonic-net#3770) (4 days ago) [Xincun Li]
* 3c50deeb - Fix `vnet_route_check` for active and inactive routes, add `--all` option (sonic-net#3763) (9 days ago) [mramezani95]
* 40ba225f - Revert "enable pfcwd for backplane ports (sonic-net#3759)" (sonic-net#3767) (13 days ago) [Dashuai Zhang]
* 2866ccd9 - enable pfcwd for backplane ports (sonic-net#3759) (2 weeks ago) [Dashuai Zhang]
* 3abd19e3 - [FC] remove FC delay field (sonic-net#3577) (3 weeks ago) [Stepan Blyshchak]
* 11c27163 - Improve SONiC disk checker to handle disk full case and mount overlay fs to allow remote user login. (sonic-net#3700) (3 weeks ago) [Hua Liu]
* 13619aa2 - [QOS] Skip showing unnecessary warning message (sonic-net#3708) (3 weeks ago) [Vivek]
* f4e6e5b8 - Fixing 'show ip bgp neighbor <ip>' in frr unified config mode (sonic-net#3738) (3 weeks ago) [kalash-nexthop]
* 9a18155e - Optimize lag_keepalive by crafting the LACPDU packet ourselves (sonic-net#3170) (3 weeks ago) [Saikrishna Arcot]
* a7deb8c0 - display proper message with proper errno for kvm. (sonic-net#3750) (3 weeks ago) [Dawei Huang]
* 865f196d - [Mellanox] Add new SN5640 platform and HwSKU (sonic-net#3742) (4 weeks ago) [Noa Or]
* 5fa85027 - [show_techsupport][pstore] Archive /var/lib/systemd/pstore info to techsupport (sonic-net#3745) (4 weeks ago) [Marty Y. Lok]
* 8f69d5a6 - sonic-utilities: WRED stats feature changes on sonic-utilities (sonic-net#2807) (4 weeks ago) [Rajesh Perumal R]
* 9d273f16 - Fix call for spanning-tree commands in dump script (sonic-net#3723) (4 weeks ago) [DavidZagury]
* 6d95d9bd - Make 'show ip bgp summary' work even when we don't have any peer groups (sonic-net#3739) (4 weeks ago) [kalash-nexthop]
* 7f3957c4 - Fix ssdhealth failure on VS platform (sonic-net#3743) (4 weeks ago) [Vivek]
* 97b4e4b8 - Fix show interface counters for Chassis Packet Supervisor (sonic-net#3734) (4 weeks ago) [anamehra]
* 414935b7 - Add recover asic_id config if load golden config. (sonic-net#3711) (5 weeks ago) [Xincun Li]
* 973cfdc3 - Remove partially installer image when image install failed. (sonic-net#3712) (5 weeks ago) [Hua Liu]
* 97c20ccd - CLI support for SmartSwitch PMON (sonic-net#3271) (5 weeks ago) [rameshraghupathy]
* 752c3d49 - [ACL] Display rule and table info written to APP DB (sonic-net#3713) (5 weeks ago) [Vivek]
* fbd0c3b1 - [show][interface] Add changes for show interface flap command (sonic-net#3724) (5 weeks ago) [vdahiya12]
* ddccabee - [show][interface] Add changes for show interface errors command (sonic-net#3721) (5 weeks ago) [vdahiya12]
* be870a6e - [config] Exit with non-zero when qos reload fail (sonic-net#3710) (6 weeks ago) [Jianyue Wu]
* ffa66e96 - Remove debug dump import by default (sonic-net#3715) (6 weeks ago) [Vivek]
* fc3a3cbe - [ssdhealth] Check for default device before falling back to discovery (sonic-net#3693) (6 weeks ago) [Vivek]
* 782c33a1 - [yang] Enforce yang full support in full config command (sonic-net#3716) (6 weeks ago) [jingwenxie]
* a5b7a904 - [show][interfaces] Add proposal for show interfaces flap (sonic-net#3627) (6 weeks ago) [vdahiya12]
* 0083a124 - sonic-utilities: add th5 hwskus to gcu conf file (sonic-net#3714) (7 weeks ago) [Chris]
* 81cf04f7 - [Mellanox] Add Mellanox-SN5610N-C256S2, Mellanox-SN5610N-C224O8 to GCU validators (sonic-net#3658) (7 weeks ago) [noaOrMlnx]
* 349a1018 - [db_migrator] Migrate tunnel table (sonic-net#3704) (7 weeks ago) [Longxiang Lyu]
* 0e327c5f - show ip interfaces: fix exception with BGP unnumbered (sonic-net#3695) (8 weeks ago) [Brad House]
* 7100f73b - [show_tech] modify generate_dump to includes BERT data and also the platform specified hw-mgmt info (sonic-net#3676) (8 weeks ago) [Marty Y. Lok]
* 80d46988 - Delete TRSNCEIVER tables while config reload (sonic-net#3680) (8 weeks ago) [noaOrMlnx]
* bf6ff9f9 - Enable multi asic golden config YANG validation. (sonic-net#3685) (8 weeks ago) [Xincun Li]
```
#### How I did it
#### How to verify it
#### Description for the changelog
…tomatically (sonic-net#21860)

#### Why I did it
src/sonic-linux-kernel
```
* 7faad28 - (HEAD -> master, origin/master, origin/HEAD) Revert "[optoe] Reset page select byte to 0 before upper memory access on pag…" (sonic-net#467) (5 days ago) [Prince George]
```
#### How I did it
#### How to verify it
#### Description for the changelog
…rms (sonic-net#21763)

- Why I did it
To return the correct Model Number for MLNX platforms rather than Part Number.

- How I did it
For SPC1-3 read it from eeprom.part_number and for SPC4 and newer read it from VPD file, SYS_DISPLAY value.

- How to verify it
Manual testing: 'show platform summary' for each spectrum.

* Modify way of getting sku number

* fix mock in test_eeprom

* Minor fix

* Add exception handling
…net#21865)

With the current fap initialization sequence, multiple faps in a modular system could come up with stand alone mode. This is not a recommended configuration. It might mess up the global clock and might ends up with traffic issue.
sonic-net#21866

This change adds a required minimum number of links have to be ready before enabling traffic on a fap, which based on the guidance should be able to guarantee the faps in a modular system can form a proper topology.
…de (sonic-net#21805)

- Why I did it
Fixing route update on duplicate link up event in promiscuous mode. When running tcpdump on interfaces which have BGP neighbors a duplicate link up event is received as the interface enters promiscuous mode. However these shouldn't trigger route updates. Due to the commit 58592be57783a3b24e7351af2a5afc61299768df route updates are published.

- How I did it
Reverted the commit that causes the issue and added as patch

- How to verify it
Running tcpdump on bgp neighbor and verifying if there are no route updates.
The latest version of setuptools-scm i.e. 8.2.0 does not support bdist_wheel. Hence there is a failure in target docker-syncd-rpc.gz due to ptf-py3 build failure. On further investigation found that bookworm installs 8.0.4 version of this package. Hence added this to the current Dockerfile.j2 of sonic-slave-bullseye. This will make it consistent with the earlier build process for ptf-py3 which used 8.1.0 version of setuptools-scm and also the sonic-slave-bookworm

Why I did it
To fix build failure of ptf-py3 on Bullseye

How I did it
Investigated the build failure and found that the recent update i.e. 8.2.0 of setuptools-scm python package does not support bdist_wheel

How to verify it
Built ptf-py3 and subsequently docker-syncd-rpc.gz with these changes. After the build sanity testing also has been done,
…mmand groups (sonic-net#21889)

Fix sonic-net#21829

Previously, Bgpcfgd generated groups of BGP configuration commands that did not have proper terminating commands. As a consequence, non-BGP configuration commands following the BGP configuration command may be rejected by the FRR because of FRR's context was not properly switched. You check the details in Issue sonic-net#21829.
…net#21882)

Why I did it
when onboard sonic-package-management extension.
It missed to add sonic version label on docker images.

Work item tracking
Microsoft ADO (number only): 25662623

How I did it
Check build logs. label is added with this PR.
…D automatically (sonic-net#21896)

#### Why I did it
src/sonic-platform-daemons
```
* 803aae7 - (HEAD -> master, origin/master, origin/HEAD) Have sff_mgr bring a linkup port out of lpmode (sonic-net#592) (50 minutes ago) [arista-nwolfe]
* ee9da5f - Refactor port_port_sfp_info_to_db() to read key:value pairs directly from port_info_dict (sonic-net#590) (3 days ago) [Vasundhara Volam]
```
#### How I did it
#### How to verify it
#### Description for the changelog
… automatically (sonic-net#21895)

#### Why I did it
src/sonic-platform-common
```
* c6f6bed - (HEAD -> master, origin/master, origin/HEAD) Restore optoe page to default 0 for active cables (sonic-net#548) (9 hours ago) [Prince George]
* d7f22a7 - Initialize unsupported values to 'N/A' in CmisApi::get_transceiver_info (sonic-net#545) (3 days ago) [Vasundhara Volam]
```
#### How I did it
#### How to verify it
#### Description for the changelog
…atically (sonic-net#21898)

#### Why I did it
src/sonic-utilities
```
* 940ed0dd - (HEAD -> master, origin/master, origin/HEAD) Fix command docker ps showing issue. (sonic-net#3791) (16 minutes ago) [Liu Shilong]
* a8ec74dd - Add namespace support for show interface neighbor expected (sonic-net#3781) (19 hours ago) [Austin (Thang Pham)]
* 8fdea8db - Fix the interface name length validation for subinterface (sonic-net#3774) (2 days ago) [Sudharsan Dhamal Gopalarathnam]
```
#### How I did it
#### How to verify it
#### Description for the changelog
…tHub release. (sonic-net#21893)

[nvidia-bluefield] Fix the issue with the SDK compilation from the GitHub release.
…#21922)

sonicbuildimage: Update openssh to openssh_9.2p1-2+deb12u5
Add mgmt_type into device metadata YANG model.
Why I did it
This PR adds tornado Tornado WSGI package to docker-ptf image. This will enable replacing the Flask based ExaBGP process API with tornado-based application. Tornado handles large number of requests with higher payload sizes automatically while Flask+Werkzeug requires additional setting changes which can be avoided keeping the code cleaner.

Work item tracking
Microsoft ADO (number only): 31250273
How I did it
Added tornado package to the docker-ptf image.

How to verify it
Ran add, remove topology to check announce routes
Ran route stress test to verify it is stable

Which release branch to backport (provide reason below if selected)
 201811
 201911
 202006
 202012
 202106
 202111
 202205
 202211
 202305
Tested branch (Please provide the tested image version)
NA

Description for the changelog
[docker-ptf] Add Tornado WSGI

This PR adds tornado Tornado WSGI package to docker-ptf image. This will enable replacing the Flask based ExaBGP process API with tornado-based application. Tornado handles large number of requests with higher payload sizes automatically while Flask+Werkzeug requires additional setting changes which can be avoided keeping the code cleaner.
…st cases

(2) Fix sample data name constraints
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@miatttao miatttao closed this Mar 5, 2025
@miatttao miatttao deleted the mia-add-young-models branch March 5, 2025 17:38
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.