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

[EVPN][yang]VRF yang model doesn't contain vni field #13456

Closed
dgsudharsan opened this issue Jan 20, 2023 · 4 comments
Closed

[EVPN][yang]VRF yang model doesn't contain vni field #13456

dgsudharsan opened this issue Jan 20, 2023 · 4 comments
Assignees
Labels
BRCM Triaged this issue has been triaged YANG YANG model related changes

Comments

@dgsudharsan
Copy link
Collaborator

Description

VRF yang model doesn't contain VNI field. The VNI field is required for EVPN configuration.
https://github.com/sonic-net/sonic-buildimage/blob/master/src/sonic-yang-models/yang-models/sonic-vrf.yang

The VRF table schema as per EVPN HLD is below
https://github.com/sonic-net/SONiC/blob/master/doc/vxlan/EVPN/EVPN_VXLAN_HLD.md

;Existing table
;defines virtual routing forward table. Updated to stores VNI associated with VRF
;
;Status: stable

key = VRF_TABLE:VRF_NAME ;
fallback = "true"/"false"
vni = 1*8DIGIT ; VNI associated with VRF

Steps to reproduce the issue:

Describe the results you received:

VRF table doesn't contain vni

Describe the results you expected:

VRF table should contain vni field

Output of show version:

(paste your output here)

Output of show techsupport:

(paste your output here or download and attach the file here )

Additional information you deem important (e.g. issue happens only occasionally):

@dgsudharsan dgsudharsan added the YANG YANG model related changes label Jan 20, 2023
@dgsudharsan
Copy link
Collaborator Author

@srj102 @adyeung @zhangyanzhao FYI

@adyeung adyeung added the BRCM label Jan 31, 2023
@gechiang gechiang added the Triaged this issue has been triaged label Feb 1, 2023
@adyeung
Copy link
Collaborator

adyeung commented Feb 1, 2023

@tapashdas to post VNI support in sonic-vrf.yang

@tapashdas
Copy link
Contributor

ETA : 02/17

tapashdas added a commit to tapashdas/sonic-buildimage that referenced this issue Feb 9, 2023
liat-grozovik pushed a commit that referenced this issue Feb 23, 2023
Why I did it
Added vni field in VRF Yang for VxLAN L3 VNI Support.

The VRF table schema as per EVPN HLD is below
https://github.com/sonic-net/SONiC/blob/master/doc/vxlan/EVPN/EVPN_VXLAN_HLD.md

Addresses Issue #13456
dgsudharsan pushed a commit to dgsudharsan/sonic-buildimage that referenced this issue Feb 24, 2023
…onic-net#13735)

Why I did it
Added vni field in VRF Yang for VxLAN L3 VNI Support.

The VRF table schema as per EVPN HLD is below
https://github.com/sonic-net/SONiC/blob/master/doc/vxlan/EVPN/EVPN_VXLAN_HLD.md

Addresses Issue sonic-net#13456
StormLiangMS pushed a commit that referenced this issue Feb 28, 2023
Manual cherry-pick of #13735
Why I did it
Added vni field in VRF Yang for VxLAN L3 VNI Support.

The VRF table schema as per EVPN HLD is below
https://github.com/sonic-net/SONiC/blob/master/doc/vxlan/EVPN/EVPN_VXLAN_HLD.md

Addresses Issue #13456
@zhangyanzhao
Copy link
Collaborator

PR was merged and close this issue.

StormLiangMS pushed a commit to StormLiangMS/sonic-buildimage that referenced this issue Mar 28, 2023
Related work items: sonic-net#276, sonic-net#305, sonic-net#332, sonic-net#338, sonic-net#339, sonic-net#1188, sonic-net#1192, sonic-net#1197, sonic-net#1206, sonic-net#1685, sonic-net#1690, sonic-net#1696, sonic-net#1699, sonic-net#1709, sonic-net#1727, sonic-net#1737, sonic-net#1741, sonic-net#1742, sonic-net#2511, sonic-net#2512, sonic-net#2532, sonic-net#2559, sonic-net#2626, sonic-net#2638, sonic-net#2645, sonic-net#2649, sonic-net#2660, sonic-net#2669, sonic-net#2670, sonic-net#2678, sonic-net#10084, sonic-net#11442, sonic-net#11873, sonic-net#12047, sonic-net#12110, sonic-net#12207, sonic-net#12529, sonic-net#12678, sonic-net#13235, sonic-net#13287, sonic-net#13372, sonic-net#13395, sonic-net#13456, sonic-net#13497, sonic-net#13522, sonic-net#13545, sonic-net#13547, sonic-net#13552, sonic-net#13569, sonic-net#13572, sonic-net#13578, sonic-net#13591, sonic-net#13611, sonic-net#13647, sonic-net#13649, sonic-net#13660, sonic-net#13710, sonic-net#13716, sonic-net#13724, sonic-net#13726, sonic-net#13732, sonic-net#13735, sonic-net#13739, sonic-net#13757, sonic-net#13786, sonic-net#13792, sonic-net#13800, sonic-net#13801, sonic-net#13802, sonic-net#13805, sonic-net#13806, sonic-net#13812, sonic-net#13814, sonic-net#13822, sonic-net#13831, sonic-net#13834, sonic-net#13847, sonic-net#13870, sonic-net#13882, sonic-net#13884, sonic-net#13885, sonic-net#13894, sonic-net#13895, sonic-net#13926, sonic-net#13932, sonic-net#13935, sonic-net#13942, sonic-net#13951, sonic-net#13953, sonic-net#13964
@dgsudharsan dgsudharsan changed the title [yang]VRF yang model doesn't contain vni field [EVPN][yang]VRF yang model doesn't contain vni field May 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BRCM Triaged this issue has been triaged YANG YANG model related changes
Projects
None yet
Development

No branches or pull requests

5 participants