]> git.puffer.fish Git - mirror/frr.git/commitdiff
tests: Fix for multicast_pim_bsm_topo1 key error 11524/head
authorKuldeep Kashyap <kashyapk@vmware.com>
Tue, 5 Jul 2022 01:01:28 +0000 (18:01 -0700)
committerKuldeep Kashyap <kashyapk@vmware.com>
Tue, 5 Jul 2022 06:00:31 +0000 (23:00 -0700)
Issue was reported by Donald, we were hitting
with key not found error and execution was
stopped, which is fixed by this PR.

Signed-off-by: Kuldeep Kashyap <kashyapk@vmware.com>
tests/topotests/lib/common_config.py

index 0e9e0ba40386db465deb8c7ecc005b77df4e76a2..4afa86f740137f617fa128ad1ecd4c3c8df51d42 100644 (file)
@@ -3437,8 +3437,23 @@ def verify_rib(
                                 found_hops = [
                                     rib_r["ip"]
                                     for rib_r in rib_routes_json[st_rt][0]["nexthops"]
+                                    if "ip" in rib_r
                                 ]
 
+                                # If somehow key "ip" is not found in nexthops JSON
+                                # then found_hops would be 0, this particular
+                                # situation will be handled here
+                                if not len(found_hops):
+                                    errormsg = (
+                                        "Nexthop {} is Missing for "
+                                        "route {} in RIB of router {}\n".format(
+                                            next_hop,
+                                            st_rt,
+                                            dut,
+                                        )
+                                    )
+                                    return errormsg
+
                                 # Check only the count of nexthops
                                 if count_only:
                                     if len(next_hop) == len(found_hops):