Merge pull request #1421 from Tristou27/iosxr_fix_1310

add outgoing_interface in get_route_to for isoxr
napalm-automation · Apr 30, 2021 · b2ee14f · b2ee14f
2 parents 5977b14 + e984d98
commit b2ee14f
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/napalm/iosxr/iosxr.py b/napalm/iosxr/iosxr.py
@@ -1857,10 +1857,17 @@ def get_route_to(self, destination="", protocol="", longer=False):
                 for route_entry in route.xpath("RoutePath/Entry"):
                     # get all possible entries
                     next_hop = napalm.base.helpers.find_txt(route_entry, "Address")
+                    outgoing_interface = napalm.base.helpers.find_txt(
+                        route_entry, "InterfaceName"
+                    )
                     single_route_details = {}
                     single_route_details.update(route_details)
                     single_route_details.update(
-                        {"current_active": first_route, "next_hop": next_hop}
+                        {
+                            "current_active": first_route,
+                            "next_hop": next_hop,
+                            "outgoing_interface": outgoing_interface,
+                        }
                     )
                     routes[destination].append(single_route_details)
                     first_route = False

diff --git a/test/iosxr/mocked_data/test_get_route_to/SR638170159/expected_result.json b/test/iosxr/mocked_data/test_get_route_to/SR638170159/expected_result.json
@@ -1 +1 @@
-{"1.0.4.0/24": [{"protocol": "bgp", "outgoing_interface": "", "age": 1804805, "current_active": true, "routing_table": "default", "last_active": false, "protocol_attributes": {}, "next_hop": "37.49.232.13", "selected_next_hop": false, "inactive_reason": "", "preference": 12}]}
+{"1.0.4.0/24": [{"protocol": "bgp", "outgoing_interface": "None", "age": 1804805, "current_active": true, "routing_table": "default", "last_active": false, "protocol_attributes": {}, "next_hop": "37.49.232.13", "selected_next_hop": false, "inactive_reason": "", "preference": 12}]}
Original file line number	Diff line number	Diff line change
		@@ -1 +1 @@
		{"1.0.4.0/24": [{"protocol": "bgp", "outgoing_interface": "", "age": 1804805, "current_active": true, "routing_table": "default", "last_active": false, "protocol_attributes": {}, "next_hop": "37.49.232.13", "selected_next_hop": false, "inactive_reason": "", "preference": 12}]}
		{"1.0.4.0/24": [{"protocol": "bgp", "outgoing_interface": "None", "age": 1804805, "current_active": true, "routing_table": "default", "last_active": false, "protocol_attributes": {}, "next_hop": "37.49.232.13", "selected_next_hop": false, "inactive_reason": "", "preference": 12}]}