Skip to content
This repository was archived by the owner on Mar 1, 2023. It is now read-only.

Commit adf1990

Browse files
authored
[yang]: Add admin_status to BGP_NEIGHBOR_TEMPLATE_LIST. (sonic-net#9341)
Add admin_status to BGP_NEIGHBOR_TEMPLATE_LIST. Signed-off-by: Gang Lv [email protected]
1 parent fdd8236 commit adf1990

File tree

4 files changed

+52
-1
lines changed

4 files changed

+52
-1
lines changed

src/sonic-yang-models/tests/files/sample_config_db.json

+2-1
Original file line numberDiff line numberDiff line change
@@ -1087,7 +1087,8 @@
10871087
"local_addr": "10.0.0.2",
10881088
"name":"PEER1",
10891089
"nhopself":"0",
1090-
"rrclient":"0"
1090+
"rrclient":"0",
1091+
"admin_status":"up"
10911092
},
10921093
"default|192.168.1.1": {
10931094
"local_asn": "65200",

src/sonic-yang-models/tests/yang_model_tests/tests/bgp.json

+5
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,11 @@
2424
"desc": "Referring non-existing BGP neighbor table.",
2525
"eStrKey" : "LeafRef"
2626
},
27+
"BGP_NEIGHBOR_NEG_INVALID_ADMIN_STATUS": {
28+
"desc": "Incorrect admin status.",
29+
"eStrKey": "InvalidValue",
30+
"eStr": ["admin_status"]
31+
},
2732
"BGP_NEIGHBOR_NEG_INVALID_NAME": {
2833
"desc": "Incorrect neighbor name.",
2934
"eStrKey": "InvalidValue",

src/sonic-yang-models/tests/yang_model_tests/tests_config/bgp.json

+36
Original file line numberDiff line numberDiff line change
@@ -133,6 +133,19 @@
133133
},
134134
"sonic-bgp-neighbor:sonic-bgp-neighbor": {
135135
"sonic-bgp-neighbor:BGP_NEIGHBOR": {
136+
"BGP_NEIGHBOR_TEMPLATE_LIST": [
137+
{
138+
"neighbor": "10.0.0.1",
139+
"asn": "65200",
140+
"holdtime": "180",
141+
"keepalive": "60",
142+
"local_addr": "10.0.0.2",
143+
"name":"PEER1",
144+
"nhopself":"0",
145+
"rrclient":"0",
146+
"admin_status":"up"
147+
}
148+
],
136149
"BGP_NEIGHBOR_LIST": [
137150
{
138151
"vrf_name": "default",
@@ -434,6 +447,29 @@
434447
}
435448
},
436449

450+
"BGP_NEIGHBOR_NEG_INVALID_ADMIN_STATUS": {
451+
"sonic-bgp-global:sonic-bgp-global": {
452+
"sonic-bgp-global:BGP_GLOBALS": {
453+
"BGP_GLOBALS_LIST": [
454+
{
455+
"vrf_name": "default",
456+
"local_asn": 65001
457+
}
458+
]
459+
}
460+
},
461+
"sonic-bgp-neighbor:sonic-bgp-neighbor": {
462+
"sonic-bgp-neighbor:BGP_NEIGHBOR": {
463+
"BGP_NEIGHBOR_TEMPLATE_LIST": [
464+
{
465+
"neighbor": "11.12.13.14",
466+
"admin_status": "right"
467+
}
468+
]
469+
}
470+
}
471+
},
472+
437473
"BGP_NEIGHBOR_NEG_INVALID_NAME": {
438474
"sonic-bgp-global:sonic-bgp-global": {
439475
"sonic-bgp-global:BGP_GLOBALS": {

src/sonic-yang-models/yang-models/sonic-bgp-neighbor.yang

+9
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,10 @@ module sonic-bgp-neighbor {
77
prefix inet;
88
}
99

10+
import sonic-types {
11+
prefix stypes;
12+
}
13+
1014
import sonic-bgp-common {
1115
prefix bgpcmn;
1216
}
@@ -97,6 +101,11 @@ module sonic-bgp-neighbor {
97101
}
98102
description "Route reflector client";
99103
}
104+
105+
leaf admin_status {
106+
type stypes:admin_status;
107+
description "Admin status of BGP peer";
108+
}
100109
}
101110

102111
list BGP_NEIGHBOR_LIST {

0 commit comments

Comments
 (0)