Skip to content

Commit

Permalink
Merge pull request #17647 from LabNConsulting/fix-oper-test
Browse files Browse the repository at this point in the history
tests: enable test failure detection and fix resulting failures
  • Loading branch information
Jafaral authored Dec 24, 2024
2 parents 9ce3b14 + 764f276 commit 803ff41
Show file tree
Hide file tree
Showing 23 changed files with 625 additions and 174 deletions.
2 changes: 1 addition & 1 deletion mgmtd/mgmt_fe_adapter.c
Original file line number Diff line number Diff line change
Expand Up @@ -190,7 +190,7 @@ static void mgmt_fe_cleanup_session(struct mgmt_fe_session_ctx **sessionp)
assert(session->adapter->refcount > 1);
mgmt_fe_adapter_unlock(&session->adapter);
}

darr_free_free(session->notify_xpaths);
hash_release(mgmt_fe_sessions, session);
XFREE(MTYPE_MGMTD_FE_SESSION, session);
*sessionp = NULL;
Expand Down
49 changes: 33 additions & 16 deletions tests/topotests/mgmt_oper/oper-results/result-lib-vrf-default.json
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@
"interface": "r1-eth0",
"active": [null],
"fib": [null],
"weight":1
"weight": 1
}
]
}
Expand Down Expand Up @@ -68,7 +68,7 @@
"interface": "r1-eth0",
"active": [null],
"fib": [null],
"weight":1
"weight": 1
}
]
}
Expand Down Expand Up @@ -97,7 +97,7 @@
"interface": "r1-eth1",
"active": [null],
"fib": [null],
"weight":1
"weight": 1
}
]
}
Expand Down Expand Up @@ -126,7 +126,7 @@
"interface": "r1-eth1",
"active": [null],
"fib": [null],
"weight":1
"weight": 1
}
]
}
Expand Down Expand Up @@ -238,7 +238,9 @@
"distance": 0,
"metric": 0,
"selected": [null],
"installed": [null],
"internal-flags": 8,
"internal-status": 16,
"uptime": "rubout",
"nexthop-group": {
"id": "rubout",
Expand All @@ -250,7 +252,7 @@
"interface": "r1-eth0",
"active": [null],
"fib": [null],
"weight":1
"weight": 1
}
]
}
Expand All @@ -265,7 +267,9 @@
"distance": 0,
"metric": 0,
"selected": [null],
"installed": [null],
"internal-flags": 8,
"internal-status": 16,
"uptime": "rubout",
"nexthop-group": {
"id": "rubout",
Expand All @@ -277,7 +281,7 @@
"interface": "r1-eth0",
"active": [null],
"fib": [null],
"weight":1
"weight": 1
}
]
}
Expand All @@ -292,7 +296,9 @@
"distance": 0,
"metric": 0,
"selected": [null],
"installed": [null],
"internal-flags": 8,
"internal-status": 16,
"uptime": "rubout",
"nexthop-group": {
"id": "rubout",
Expand All @@ -304,7 +310,7 @@
"interface": "r1-eth1",
"active": [null],
"fib": [null],
"weight":1
"weight": 1
}
]
}
Expand All @@ -319,7 +325,9 @@
"distance": 0,
"metric": 0,
"selected": [null],
"installed": [null],
"internal-flags": 8,
"internal-status": 16,
"uptime": "rubout",
"nexthop-group": {
"id": "rubout",
Expand All @@ -331,7 +339,7 @@
"interface": "r1-eth1",
"active": [null],
"fib": [null],
"weight":1
"weight": 1
}
]
}
Expand Down Expand Up @@ -369,7 +377,7 @@
"interface": "r1-eth0",
"active": [null],
"fib": [null],
"weight":1
"weight": 1
}
]
}
Expand Down Expand Up @@ -398,7 +406,7 @@
"interface": "r1-eth0",
"active": [null],
"fib": [null],
"weight":1
"weight": 1
}
]
}
Expand Down Expand Up @@ -427,7 +435,7 @@
"interface": "r1-eth1",
"active": [null],
"fib": [null],
"weight":1
"weight": 1
}
]
}
Expand Down Expand Up @@ -456,7 +464,7 @@
"interface": "r1-eth1",
"active": [null],
"fib": [null],
"weight":1
"weight": 1
}
]
}
Expand All @@ -480,7 +488,9 @@
"distance": 0,
"metric": 0,
"selected": [null],
"installed": [null],
"internal-flags": 8,
"internal-status": 16,
"uptime": "rubout",
"nexthop-group": {
"id": "rubout",
Expand All @@ -492,7 +502,7 @@
"interface": "r1-eth0",
"active": [null],
"fib": [null],
"weight":1
"weight": 1
}
]
}
Expand All @@ -507,7 +517,9 @@
"distance": 0,
"metric": 0,
"selected": [null],
"installed": [null],
"internal-flags": 8,
"internal-status": 16,
"uptime": "rubout",
"nexthop-group": {
"id": "rubout",
Expand All @@ -519,7 +531,7 @@
"interface": "r1-eth0",
"active": [null],
"fib": [null],
"weight":1
"weight": 1
}
]
}
Expand All @@ -534,7 +546,9 @@
"distance": 0,
"metric": 0,
"selected": [null],
"installed": [null],
"internal-flags": 8,
"internal-status": 16,
"uptime": "rubout",
"nexthop-group": {
"id": "rubout",
Expand All @@ -546,7 +560,7 @@
"interface": "r1-eth1",
"active": [null],
"fib": [null],
"weight":1
"weight": 1
}
]
}
Expand All @@ -561,7 +575,9 @@
"distance": 0,
"metric": 0,
"selected": [null],
"installed": [null],
"internal-flags": 8,
"internal-status": 16,
"uptime": "rubout",
"nexthop-group": {
"id": "rubout",
Expand All @@ -573,7 +589,7 @@
"interface": "r1-eth1",
"active": [null],
"fib": [null],
"weight":1
"weight": 1
}
]
}
Expand All @@ -589,3 +605,4 @@
]
}
}

Loading

0 comments on commit 803ff41

Please sign in to comment.