From a28055a47830398da3b84433435df8ff4fc29e09 Mon Sep 17 00:00:00 2001 From: Donald Sharp Date: Thu, 19 Nov 2020 07:40:03 -0500 Subject: [PATCH] tests: Fix common_config.py issues 1) in generate_ips if start_ipaddr does not have a `/` in it there exists a code path where both mask and step are null values. Write a bit of code to ensure this pre-req is found early and often 2) in verify_rib there exists a code path where static_route is null when we get to the non static route section. Change the code to operate on the advertise_network_dict that we are iterating over. Signed-off-by: Donald Sharp --- tests/topotests/lib/common_config.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/tests/topotests/lib/common_config.py b/tests/topotests/lib/common_config.py index 9c104db3cd..6c24b6ddbb 100644 --- a/tests/topotests/lib/common_config.py +++ b/tests/topotests/lib/common_config.py @@ -1150,6 +1150,9 @@ def generate_ips(network, no_of_ips): if "/" in start_ipaddr: start_ip = start_ipaddr.split("/")[0] mask = int(start_ipaddr.split("/")[1]) + else: + logger.debug("start_ipaddr {} must have a / in it".format(start_ipaddr)) + assert(0) addr_type = validate_ip_address(start_ip) if addr_type == "ipv4": @@ -2865,7 +2868,7 @@ def verify_rib( for advertise_network_dict in advertise_network: if "vrf" in advertise_network_dict: - cmd = "{} vrf {} json".format(command, static_route["vrf"]) + cmd = "{} vrf {} json".format(command, advertise_network_dict["vrf"]) else: cmd = "{} json".format(command)