Skip to content

Commit

Permalink
[system_health] Fix test_system_health failed due to change of log fo…
Browse files Browse the repository at this point in the history
…rmat (#7649)

What is the motivation for this PR?
test_service_checker_with_process_exit failed due to this change: sonic-net/sonic-buildimage#13497

How did you do it?
Modify format of verfiying log.

How did you verify/test it?
Run test

Signed-off-by: Yaqiang Zhu <[email protected]>
  • Loading branch information
yaqiangz authored Mar 6, 2023
1 parent 7899108 commit 45b43c6
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tests/system_health/test_system_health.py
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,7 @@ def test_service_checker_with_process_exit(duthosts, enum_rand_one_per_hwsku_hos
# use wait_until to check if SYSTEM_HEALTH_INFO has expected content
# avoid waiting for too long or DEFAULT_INTERVAL is not long enough to refresh db
category = '{}:{}'.format(container, critical_process)
expected_value = "'{}' is not running".format(critical_process)
expected_value = "Process '{}' in container '{}' is not running".format(critical_process, container)
result = wait_until(WAIT_TIMEOUT, 10, 2, check_system_health_info, duthost, category, expected_value)
assert result == True, '{} is not recorded'.format(critical_process)
summary = redis_get_field_value(duthost, STATE_DB, HEALTH_TABLE_NAME, 'summary')
Expand Down

0 comments on commit 45b43c6

Please sign in to comment.