mirror of
https://git.proxmox.com/git/mirror_frr
synced 2025-08-06 10:54:47 +00:00
tests: Removing invalid step from ospf tests.
1. Removed the step from hello test case with hello timer of 65535. This test works in some platforms and does not work in others, affecting stability. Signed-off-by: nguggarigoud <nguggarigoud@vmware.com>
This commit is contained in:
parent
2af482767c
commit
d5bd355430
@ -596,93 +596,6 @@ def test_ospf_hello_tc10_p0(request):
|
||||
ospf_covergence
|
||||
)
|
||||
|
||||
step(" Configure hello timer = 65535")
|
||||
topo1 = {
|
||||
"r0": {
|
||||
"links": {
|
||||
"r1": {
|
||||
"interface": topo["routers"]["r0"]["links"]["r1"]["interface"],
|
||||
"ospf": {"hello_interval": 65535, "dead_interval": 4},
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
result = create_interfaces_cfg(tgen, topo1)
|
||||
assert result is True, "Testcase {} : Failed \n Error: {}".format(tc_name, result)
|
||||
|
||||
topo1 = {
|
||||
"r1": {
|
||||
"links": {
|
||||
"r0": {
|
||||
"interface": topo["routers"]["r1"]["links"]["r0"]["interface"],
|
||||
"ospf": {"hello_interval": 65535, "dead_interval": 4},
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
result = create_interfaces_cfg(tgen, topo1)
|
||||
assert result is True, "Testcase {} : Failed \n Error: {}".format(tc_name, result)
|
||||
|
||||
step("verify that new timer value is configured.")
|
||||
input_dict = {
|
||||
"r0": {
|
||||
"links": {"r1": {"ospf": {"timerMsecs": 65535 * 1000, "timerDeadSecs": 4}}}
|
||||
}
|
||||
}
|
||||
dut = "r0"
|
||||
result = verify_ospf_interface(tgen, topo, dut=dut, input_dict=input_dict)
|
||||
assert result is True, "Testcase {} : Failed \n Error: {}".format(tc_name, result)
|
||||
|
||||
step("verify that ospf neighbours are full")
|
||||
ospf_covergence = verify_ospf_neighbor(tgen, topo, dut=dut)
|
||||
assert ospf_covergence is True, "setup_module :Failed \n Error:" " {}".format(
|
||||
ospf_covergence
|
||||
)
|
||||
|
||||
step(" Try configuring timer values outside range for example 65536")
|
||||
topo1 = {
|
||||
"r0": {
|
||||
"links": {
|
||||
"r1": {
|
||||
"interface": topo["routers"]["r0"]["links"]["r1"]["interface"],
|
||||
"ospf": {"hello_interval": 65536, "dead_interval": 4},
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
result = create_interfaces_cfg(tgen, topo1)
|
||||
assert result is not True, "Testcase {} : Failed \n Error: {}".format(
|
||||
tc_name, result
|
||||
)
|
||||
|
||||
step("Unconfigure the hello timer from the interface from r1 and r2.")
|
||||
|
||||
topo1 = {
|
||||
"r1": {
|
||||
"links": {
|
||||
"r0": {
|
||||
"interface": topo["routers"]["r1"]["links"]["r0"]["interface"],
|
||||
"ospf": {"hello_interval": 65535},
|
||||
"delete": True,
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
result = create_interfaces_cfg(tgen, topo1)
|
||||
assert result is True, "Testcase {} : Failed \n Error: {}".format(tc_name, result)
|
||||
|
||||
step(
|
||||
"Verify that timer value is deleted from intf & " "set to default value 40 sec."
|
||||
)
|
||||
input_dict = {"r1": {"links": {"r0": {"ospf": {"timerMsecs": 10 * 1000}}}}}
|
||||
dut = "r1"
|
||||
result = verify_ospf_interface(tgen, topo, dut=dut, input_dict=input_dict)
|
||||
assert result is True, "Testcase {} : Failed \n Error: {}".format(tc_name, result)
|
||||
|
||||
write_test_footer(tc_name)
|
||||
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user