matthew/sonic-buildimage
Archived
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[yang]Add missing fields in PortChannel yang model (#14232)

Browse Source 
Porting fix https://github.com/sonic-net/sonic-buildimage/pull/14045 to 202205. 202205 doesn't have fast_rate and hence only fallback is updated in yang model

#### Why I did it
Added Missing fields in sonic-portchannel yang model.
"fallback" is present in configuration schema but not in yang model. This leads to traceback when yang is validated

#### How I did it
Updated yang model

#### How to verify it
Added tests to verify

#### Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

#### Link to config_db schema for YANG module changes
Part of the PR
This commit is contained in:
Sudharsan Dhamal Gopalarathnam 2023-03-14 10:24:34 -07:00 committed by GitHub
parent da348ecb79
commit 492b2b69da
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
5 changed files with 31 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/sonic-yang-models/doc/Configuration.md
View File

@ -1259,7 +1259,8 @@ name as object key and member list as attribute.
"members": [
"Ethernet56"
],
"mtu": "9100"
"mtu": "9100",
"fallback": "false"
}
}
}

3
src/sonic-yang-models/tests/files/sample_config_db.json
View File

@ -99,7 +99,8 @@
"admin_status": "up",
"min_links": "2",
"mtu": "9100",
"tpid": "0x8100"
"tpid": "0x8100",
"fallback" : "true"
}
},
"PORTCHANNEL_INTERFACE": {

5
src/sonic-yang-models/tests/yang_model_tests/tests/portchannel.json
View File

@ -19,6 +19,11 @@
"eStrKey" : "Pattern",
"eStr": ["0x8100|0x9100|0x9200|0x88a8|0x88A8"]
},
"PORT_CHANNEL_INVALID_FALLBACK": {
"desc": "INCORRECT PORTCHANNEL FALLBACK IN PORT_CHANNEL TABLE.",
"eStrKey" : "Pattern",
"eStr": ["false|true|False|True"]
},
"PORTCHANNEL_INTERFACE_IP_ADDR_TEST": {
"desc": "Configure IP address on PORTCHANNEL_INTERFACE table."
},

18
src/sonic-yang-models/tests/yang_model_tests/tests_config/portchannel.json
View File

@ -25,7 +25,8 @@
"mtu": "9100",
"tpid": "0x8100",
"lacp_key": "auto",
"name": "PortChannel0001"
"name": "PortChannel0001",
"fallback" : "false"
}
]
}
@ -116,6 +117,21 @@
}
}
},
"PORT_CHANNEL_INVALID_FALLBACK": {
"sonic-portchannel:sonic-portchannel": {
"sonic-portchannel:PORTCHANNEL": {
"PORTCHANNEL_LIST": [
{
"admin_status": "up",
"mtu": "9100",
"tpid": "0x9100",
"name": "PortChannel0001",
"fallback": "enabled"
}
]
}
}
},
"PORTCHANNEL_INTERFACE_IP_ADDR_TEST": {
"sonic-portchannel:sonic-portchannel": {
"sonic-portchannel:PORTCHANNEL": {

5
src/sonic-yang-models/yang-models/sonic-portchannel.yang
View File

@ -102,6 +102,11 @@ module sonic-portchannel {
type stypes:tpid_type;
}
leaf fallback {
description "Enable LACP fallback feature";
type stypes:boolean_type;
}
} /* end of list PORTCHANNEL_LIST */
} /* end of container PORTCHANNEL */