Enhance the syslib library
Enhance syslib/utils_os.robot by
- Adding messages to the "Should Be Equal" keywords.
- Enhancing the "Is HTX Running" keyword to return status
rather than terminating the run.
- Update the systest/* tests to use the revised
"Is HTX Running" keyword.
- Display BMC error logs before termination.
Resolves openbmc/openbmc-test-automation#1304
Change-Id: I17ab8d2deafbe1ef2a232d8356e78740b1b20991
Signed-off-by: Steven Sombar <ssombar@us.ibm.com>
diff --git a/systest/gpu_stress_test.robot b/systest/gpu_stress_test.robot
index f2f9ad8..8788ad5 100755
--- a/systest/gpu_stress_test.robot
+++ b/systest/gpu_stress_test.robot
@@ -57,7 +57,7 @@
# Shutdown HTX if it is already running.
- ${status}= Run Keyword And Return Status Is HTX Running
+ ${status}= Is HTX Running
Run Keyword If '${status}' == 'True'
... Shutdown HTX Exerciser
diff --git a/systest/htx_hardbootme_test.robot b/systest/htx_hardbootme_test.robot
index c1b2fd5..853241c 100755
--- a/systest/htx_hardbootme_test.robot
+++ b/systest/htx_hardbootme_test.robot
@@ -38,8 +38,10 @@
Resource ../syslib/utils_os.robot
Resource ../lib/openbmc_ffdc_utils.robot
+Resource ../lib/logging_utils.robot
Library ../syslib/utils_keywords.py
Library ../lib/utils_files.py
+Library ../lib/logging_utils.py
Suite Setup Run Keyword Start SOL Console Logging
Test Setup Test Setup Execution
@@ -138,7 +140,13 @@
... Do Inventory And Compare ${json_final_file_path}
... ${PREV_INV_FILE_PATH}
- Error Logs Should Not Exist
+ # Terminate run if there are any BMC error logs.
+ ${error_logs}= Get Error Logs
+ ${num_logs}= Get Length ${error_logs}
+ Run Keyword If ${num_logs} != 0 Run Keywords
+ ... Print Error Logs ${error_logs}
+ ... AND Fail msg=Terminating run due to BMC error log(s).
+
Power Off Host
# Close all SSH and REST active sessions.
@@ -183,6 +191,7 @@
... ${file2} ${json_diff_file_path} ${INV_IGNORE_LIST}
Run Keyword If '${diff_rc}' != '${0}'
... Report Inventory Mismatch ${diff_rc} ${json_diff_file_path}
+ ... ELSE Rprint Timen Inventoy check: No differences found.
Report Inventory Mismatch
@@ -217,7 +226,7 @@
Tool Exist htxcmdline
# Shutdown if HTX is running.
- ${status}= Run Keyword And Return Status Is HTX Running
+ ${status}= Is HTX Running
Run Keyword If '${status}' == 'True'
... Shutdown HTX Exerciser
diff --git a/systest/network_stability_test.robot b/systest/network_stability_test.robot
index da09939..5194a71 100755
--- a/systest/network_stability_test.robot
+++ b/systest/network_stability_test.robot
@@ -82,7 +82,7 @@
Tool Exist htxcmdline
# Shutdown if HTX is running.
- ${status}= Run Keyword And Return Status Is HTX Running
+ ${status}= Is HTX Running
Run Keyword If '${status}' == 'True'
... Shutdown HTX Exerciser