Skip to content

Commit ea78527

Browse files
mirceaulinicmirceaulinic
authored
Merge pull request #1980 from tcaiazza/nxos-bgp-shut
Fix NXOS get_bgp_neighbor when neighbor is admin shut
2 parents 1014cfa + 78f5aaa commit ea78527

File tree

3 files changed

+848
-0
lines changed

3 files changed

+848
-0
lines changed

Diff for: ‎napalm/nxos/nxos.py

+1
Original file line numberDiff line numberDiff line change
@@ -1046,6 +1046,7 @@ def get_bgp_neighbors(self) -> Dict[str, models.BGPStateNeighborsPerVRFDict]:
10461046
"Established": {"is_up": True, "is_enabled": True},
10471047
"Closing": {"is_up": True, "is_enabled": True},
10481048
"Shutdown": {"is_up": False, "is_enabled": False},
1049+
"Admin (Shut)": {"is_up": False, "is_enabled": False},
10491050
}
10501051
"""
10511052
af_name_dict = {

Diff for: ‎test/nxos/mocked_data/test_get_bgp_neighbors/issue_870/expected_result.json

+265
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,265 @@
1+
{
2+
"RED-PROD": {
3+
"router_id": "10.2.3.6",
4+
"peers": {
5+
"10.1.1.169": {
6+
"is_enabled": true,
7+
"address_family": {
8+
"ipv4": {
9+
"sent_prefixes": -1,
10+
"accepted_prefixes": -1,
11+
"received_prefixes": 4
12+
}
13+
},
14+
"uptime": -1,
15+
"remote_as": 64862,
16+
"description": "",
17+
"remote_id": "10.1.1.169",
18+
"local_as": 64861,
19+
"is_up": true
20+
},
21+
"10.1.1.171": {
22+
"is_enabled": true,
23+
"address_family": {
24+
"ipv4": {
25+
"sent_prefixes": -1,
26+
"accepted_prefixes": -1,
27+
"received_prefixes": 4
28+
}
29+
},
30+
"uptime": -1,
31+
"remote_as": 64862,
32+
"description": "",
33+
"remote_id": "10.1.1.171",
34+
"local_as": 64861,
35+
"is_up": true
36+
}
37+
}
38+
},
39+
"BLUE": {
40+
"router_id": "10.1.1.186",
41+
"peers": {
42+
"10.1.1.187": {
43+
"is_enabled": true,
44+
"address_family": {
45+
"ipv4": {
46+
"sent_prefixes": -1,
47+
"accepted_prefixes": -1,
48+
"received_prefixes": 0
49+
}
50+
},
51+
"uptime": -1,
52+
"remote_as": 64862,
53+
"description": "",
54+
"remote_id": "10.1.1.187",
55+
"local_as": 64861,
56+
"is_up": true
57+
},
58+
"10.1.1.185": {
59+
"is_enabled": true,
60+
"address_family": {
61+
"ipv4": {
62+
"sent_prefixes": -1,
63+
"accepted_prefixes": -1,
64+
"received_prefixes": 0
65+
}
66+
},
67+
"uptime": -1,
68+
"remote_as": 64862,
69+
"description": "",
70+
"remote_id": "10.1.1.185",
71+
"local_as": 64861,
72+
"is_up": true
73+
}
74+
}
75+
},
76+
"MGMT": {
77+
"router_id": "10.0.0.188",
78+
"peers": {
79+
"10.1.1.227": {
80+
"is_enabled": true,
81+
"address_family": {
82+
"ipv4": {
83+
"sent_prefixes": -1,
84+
"accepted_prefixes": -1,
85+
"received_prefixes": 14
86+
}
87+
},
88+
"uptime": -1,
89+
"remote_as": 64862,
90+
"description": "",
91+
"remote_id": "10.1.1.227",
92+
"local_as": 64861,
93+
"is_up": true
94+
},
95+
"10.1.1.225": {
96+
"is_enabled": true,
97+
"address_family": {
98+
"ipv4": {
99+
"sent_prefixes": -1,
100+
"accepted_prefixes": -1,
101+
"received_prefixes": 14
102+
}
103+
},
104+
"uptime": -1,
105+
"remote_as": 64862,
106+
"description": "",
107+
"remote_id": "10.1.1.225",
108+
"local_as": 64861,
109+
"is_up": true
110+
}
111+
}
112+
},
113+
"global": {
114+
"router_id": "10.3.1.136",
115+
"peers": {
116+
"10.1.1.135": {
117+
"is_enabled": true,
118+
"address_family": {
119+
"l2vpn": {
120+
"sent_prefixes": -1,
121+
"accepted_prefixes": -1,
122+
"received_prefixes": 7237
123+
}
124+
},
125+
"uptime": -1,
126+
"remote_as": 64861,
127+
"description": "",
128+
"remote_id": "10.1.1.135",
129+
"local_as": 64861,
130+
"is_up": true
131+
},
132+
"10.1.1.134": {
133+
"is_enabled": true,
134+
"address_family": {
135+
"l2vpn": {
136+
"sent_prefixes": -1,
137+
"accepted_prefixes": -1,
138+
"received_prefixes": 7237
139+
}
140+
},
141+
"uptime": -1,
142+
"remote_as": 64861,
143+
"description": "",
144+
"remote_id": "10.1.1.134",
145+
"local_as": 64861,
146+
"is_up": true
147+
}
148+
}
149+
},
150+
"RED-PRE-PROD": {
151+
"router_id": "10.2.63.60",
152+
"peers": {
153+
"10.1.1.179": {
154+
"is_enabled": true,
155+
"address_family": {
156+
"ipv4": {
157+
"sent_prefixes": -1,
158+
"accepted_prefixes": -1,
159+
"received_prefixes": 13
160+
}
161+
},
162+
"uptime": -1,
163+
"remote_as": 64862,
164+
"description": "",
165+
"remote_id": "10.1.1.179",
166+
"local_as": 64861,
167+
"is_up": true
168+
},
169+
"10.1.1.177": {
170+
"is_enabled": false,
171+
"address_family": {
172+
"ipv4": {
173+
"sent_prefixes": -1,
174+
"accepted_prefixes": -1,
175+
"received_prefixes": 0
176+
}
177+
},
178+
"uptime": -1,
179+
"remote_as": 64862,
180+
"description": "",
181+
"remote_id": "10.1.1.177",
182+
"local_as": 64861,
183+
"is_up": false
184+
}
185+
}
186+
},
187+
"NON_PROD": {
188+
"router_id": "10.1.255.188",
189+
"peers": {
190+
"10.1.1.161": {
191+
"is_enabled": true,
192+
"address_family": {
193+
"ipv4": {
194+
"sent_prefixes": -1,
195+
"accepted_prefixes": -1,
196+
"received_prefixes": 441
197+
}
198+
},
199+
"uptime": -1,
200+
"remote_as": 64862,
201+
"description": "",
202+
"remote_id": "10.1.1.161",
203+
"local_as": 64861,
204+
"is_up": true
205+
},
206+
"10.1.1.163": {
207+
"is_enabled": true,
208+
"address_family": {
209+
"ipv4": {
210+
"sent_prefixes": -1,
211+
"accepted_prefixes": -1,
212+
"received_prefixes": 441
213+
}
214+
},
215+
"uptime": -1,
216+
"remote_as": 64862,
217+
"description": "",
218+
"remote_id": "10.1.1.163",
219+
"local_as": 64861,
220+
"is_up": true
221+
}
222+
}
223+
},
224+
"PROD": {
225+
"router_id": "10.9.2.60",
226+
"peers": {
227+
"10.1.1.153": {
228+
"is_enabled": true,
229+
"address_family": {
230+
"ipv4": {
231+
"sent_prefixes": -1,
232+
"accepted_prefixes": -1,
233+
"received_prefixes": 9
234+
}
235+
},
236+
"uptime": -1,
237+
"remote_as": 64862,
238+
"description": "",
239+
"remote_id": "10.1.1.153",
240+
"local_as": 64861,
241+
"is_up": true
242+
},
243+
"10.1.1.155": {
244+
"is_enabled": true,
245+
"address_family": {
246+
"ipv4": {
247+
"sent_prefixes": -1,
248+
"accepted_prefixes": -1,
249+
"received_prefixes": 9
250+
}
251+
},
252+
"uptime": -1,
253+
"remote_as": 64862,
254+
"description": "",
255+
"remote_id": "10.1.1.155",
256+
"local_as": 64861,
257+
"is_up": true
258+
}
259+
}
260+
},
261+
"OTV": {
262+
"router_id": "10.3.1.228",
263+
"peers": {}
264+
}
265+
}

0 commit comments

Comments
 (0)