IPMI SOL log file path default
Changes:
- Default the SOL in user execution directory logs/
- Update Power on and off to use IPMI methods
This changes is to avoid leaving files under /tmp/ which will
eventually flush on the reboot but user using server running
the test hardly reboot the system. The existing scheme works
ok if running on local or docker instances but still not a
good idea to dump onto /tmp/.
Resolves openbmc/openbmc-test-automation#2132
Change-Id: Ia699f7faf0df1badc5fcd37b638dc0008ae2bc5f
Signed-off-by: George Keishing <gkeishin@in.ibm.com>
diff --git a/ipmi/test_ipmi_sol.robot b/ipmi/test_ipmi_sol.robot
index 199063b..fbb51d0 100644
--- a/ipmi/test_ipmi_sol.robot
+++ b/ipmi/test_ipmi_sol.robot
@@ -127,7 +127,7 @@
[Documentation] Verify SOL during boot.
[Tags] Verify_SOL_During_Boot
- Redfish Hard Power Off stack_mode=skip
+ IPMI Power Off stack_mode=skip
Activate SOL Via IPMI
Initiate Host Boot Via External IPMI wait=${0}
@@ -137,7 +137,7 @@
Wait Until Keyword Succeeds 3 mins 15 secs
... Check IPMI SOL Output Content ISTEP
- Redfish Hard Power Off
+ IPMI Power Off
Verify Deactivate Non Existing SOL
@@ -200,11 +200,11 @@
Check IPMI SOL Output Content
[Documentation] Check if SOL has given content.
- [Arguments] ${data} ${file_path}=/tmp/sol_${OPENBMC_HOST}
+ [Arguments] ${data} ${file_path}=${IPMI_SOL_LOG_FILE}
# Description of argument(s):
# data Content which need to be checked(e.g. Petitboot, ISTEP).
# file_path The file path on the local machine to check SOL content.
- # By default it check SOL content from /tmp/sol_<BMC_IP>.
+ # By default it check SOL content from log/sol_<BMC_IP>.
${output}= OperatingSystem.Get File ${file_path} encoding_errors=ignore
Should Contain ${output} ${data} case_insensitive=True
@@ -249,7 +249,7 @@
Should Be Equal '${setting_value}' '${expected_value}'
- Redfish Hard Power Off stack_mode=skip
+ IPMI Power Off stack_mode=skip
Initiate Host Boot Via External IPMI wait=${0}