Skip to content

Commit 8f36015

Browse files
justin-wong-cevvolam
authored andcommitted
Add config change validation to prevent bad bgp config state (sonic-net#19139)
What is the motivation for this PR? bgp/test_bgp_suppress_fib.py::test_suppress_fib_stress failing How did you do it? Wait for config to properly take effect bgp/test_bgp_suppress_fib.py::test_credit_loop so subsequent tests can run on a clean state. How did you verify/test it? bgp/test_bgp_suppress_fib.py::test_suppress_fib_stress no longer has packet count mismatch failures.
1 parent 81b8a66 commit 8f36015

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

tests/bgp/test_bgp_suppress_fib.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1052,7 +1052,7 @@ def test_credit_loop(duthost, tbinfo, nbrhosts, ptfadapter, prepare_param, gener
10521052
for exabgp_port, exabgp_port_v6, recv_port in zip(exabgp_port_list, exabgp_port_list_v6, recv_port_list):
10531053
try:
10541054
with allure.step("Disable bgp suppress-fib-pending function"):
1055-
config_bgp_suppress_fib(duthost, False)
1055+
config_bgp_suppress_fib(duthost, False, validate_result=True)
10561056

10571057
with allure.step(
10581058
"Validate traffic is forwarded back to T2 VM and routes in HW table are removed by orchagent"):

0 commit comments

Comments
 (0)