Skip to content

Commit 8f279a1

Browse files
jipanyangqiluo-msft
authored andcommitted
Switch to "config warm_restart" CLI for warm restart configuration (sonic-net#633)
1 parent f13aaed commit 8f279a1

File tree

1 file changed

+6
-47
lines changed

1 file changed

+6
-47
lines changed

tests/test_warm_reboot.py

+6-47
Original file line numberDiff line numberDiff line change
@@ -129,15 +129,7 @@ def test_PortSyncdWarmRestart(dvs):
129129
appl_db = swsscommon.DBConnector(swsscommon.APPL_DB, dvs.redis_sock, 0)
130130
state_db = swsscommon.DBConnector(swsscommon.STATE_DB, dvs.redis_sock, 0)
131131

132-
# enable warm restart
133-
# TODO: use cfg command to config it
134-
create_entry_tbl(
135-
conf_db,
136-
swsscommon.CFG_WARM_RESTART_TABLE_NAME, "swss",
137-
[
138-
("enable", "true"),
139-
]
140-
)
132+
dvs.runcmd("config warm_restart enable swss")
141133

142134
dvs.runcmd("ifconfig Ethernet16 up")
143135
dvs.runcmd("ifconfig Ethernet20 up")
@@ -220,15 +212,7 @@ def test_VlanMgrdWarmRestart(dvs):
220212

221213
time.sleep(1)
222214

223-
# enable warm restart
224-
# TODO: use cfg command to config it
225-
create_entry_tbl(
226-
conf_db,
227-
swsscommon.CFG_WARM_RESTART_TABLE_NAME, "swss",
228-
[
229-
("enable", "true"),
230-
]
231-
)
215+
dvs.runcmd("config warm_restart enable swss")
232216

233217
# create vlan
234218
create_entry_tbl(
@@ -287,7 +271,6 @@ def test_VlanMgrdWarmRestart(dvs):
287271
(status, fvs) = tbl.get("Vlan20:11.0.0.10")
288272
assert status == True
289273

290-
291274
(exitcode, bv_before) = dvs.runcmd("bridge vlan")
292275
print(bv_before)
293276

@@ -367,15 +350,7 @@ def test_swss_neighbor_syncup(dvs):
367350
conf_db = swsscommon.DBConnector(swsscommon.CONFIG_DB, dvs.redis_sock, 0)
368351
state_db = swsscommon.DBConnector(swsscommon.STATE_DB, dvs.redis_sock, 0)
369352

370-
# enable warm restart
371-
# TODO: use cfg command to config it
372-
create_entry_tbl(
373-
conf_db,
374-
swsscommon.CFG_WARM_RESTART_TABLE_NAME, "swss",
375-
[
376-
("enable", "true"),
377-
]
378-
)
353+
dvs.runcmd("config warm_restart enable swss")
379354

380355
#
381356
# Testcase1:
@@ -607,13 +582,7 @@ def test_swss_neighbor_syncup(dvs):
607582
# setup timer in configDB
608583
timer_value = "15"
609584

610-
create_entry_tbl(
611-
conf_db,
612-
swsscommon.CFG_WARM_RESTART_TABLE_NAME, "swss",
613-
[
614-
("neighsyncd_timer", timer_value),
615-
]
616-
)
585+
dvs.runcmd("config warm_restart neighsyncd_timer {}".format(timer_value))
617586

618587
# get restore_count
619588
restore_count = swss_get_RestoreCount(state_db)
@@ -705,9 +674,7 @@ def test_OrchagentWarmRestartReadyCheck(dvs):
705674
dvs.runcmd("ip -s -s neigh flush all")
706675
time.sleep(1)
707676

708-
# enable warm restart
709-
# TODO: use cfg command to config it
710-
conf_db = swsscommon.DBConnector(swsscommon.CONFIG_DB, dvs.redis_sock, 0)
677+
dvs.runcmd("config warm_restart enable swss")
711678

712679
dvs.runcmd("ifconfig Ethernet0 10.0.0.0/31 up")
713680
dvs.runcmd("ifconfig Ethernet4 10.0.0.2/31 up")
@@ -757,15 +724,7 @@ def test_swss_port_state_syncup(dvs):
757724
conf_db = swsscommon.DBConnector(swsscommon.CONFIG_DB, dvs.redis_sock, 0)
758725
state_db = swsscommon.DBConnector(swsscommon.STATE_DB, dvs.redis_sock, 0)
759726

760-
# enable warm restart
761-
# TODO: use cfg command to config it
762-
create_entry_tbl(
763-
conf_db,
764-
swsscommon.CFG_WARM_RESTART_TABLE_NAME, "swss",
765-
[
766-
("enable", "true"),
767-
]
768-
)
727+
dvs.runcmd("config warm_restart enable swss")
769728

770729
tbl = swsscommon.Table(appl_db, swsscommon.APP_PORT_TABLE_NAME)
771730

0 commit comments

Comments
 (0)