Skip to content

Commit be7bda5

Browse files
ZhaohuiSmssonicbld
authored andcommitted
Add safe_reload for config_reload in test_syslog_srouce_ip (sonic-net#16969)
What is the motivation for this PR? Without safe_reload, it's not enough time to wait system healthy. safe_reload will check critical services after reload. How did you do it? Add safe_reload=True, for config_reload How did you verify/test it? Run test_syslog_source_ip on testbed.
1 parent 218c703 commit be7bda5

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

tests/syslog/test_syslog_source_ip.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ def restore_config_by_config_reload(duthosts, enum_rand_one_per_hwsku_frontend_h
9797
state='absent', delay=1, timeout=30)
9898
localhost.wait_for(host=duthost.mgmt_ip, port=SONIC_SSH_PORT, search_regex=SONIC_SSH_REGEX,
9999
state='started', delay=2, timeout=180)
100-
config_reload(duthost)
100+
config_reload(duthost, safe_reload=True)
101101

102102

103103
@pytest.fixture(autouse=True)

0 commit comments

Comments
 (0)