fix System event log entry is not empty

Symptom:
When running full automation test cases, the test case
"Reboot And Check Ambient Temperature Records Are Valid" sometimes got failed.
The error log i

fix System event log entry is not empty

Symptom:
When running full automation test cases, the test case
"Reboot And Check Ambient Temperature Records Are Valid" sometimes got failed.
The error log is "System event log entry is not empty".

Root cause:
The keyword "Get Thermal Records and Verify" will check whether there is any
error event log when checking ambient temperature records are valid.
However, all temperature records are valid in test result.
But, seems there are other unrelated system error event logs already exist.
Thus, cause this keyword return failed.

Solution:
Prevent this unexpected redfish event log when doing some test items.
We should clear redfish event log to make sure the error event is not cause
by current test item.

Tested:
Run robot test redfish/systems/test_thermal_ambient_temperatures.robot.

Signed-off-by: Tim Lee <timlee660101@gmail.com>
Change-Id: Id1d9d9bd2307f51be67e284f64373622b147cc5c

show more ...

systems: thermal: use redfish error log checking

Symptom:
Auto test got failed from redfish/systems/test_thermal_ambient_temperatures.robot
Error is "Unexpected BMC error log(s) present.: 200 != 404

systems: thermal: use redfish error log checking

Symptom:
Auto test got failed from redfish/systems/test_thermal_ambient_temperatures.robot
Error is "Unexpected BMC error log(s) present.: 200 != 404" and result is FAIL
Get Thermal Records and Verify | FAIL |
Reboot And Check Ambient Temperature Records Are Valid | FAIL |

Root cause:
In Test Setup that already change to use redfish error log purging.
But, "Get Thermal Records and Verify" still using legacy method
"Error Logs Should Not Exist" then cause unexpected error log present.

Solution:
Using redfish error log checking "Event Log Should Not Exist" instead.

Tested:
Run robot test redfish/systems/test_thermal_ambient_temperatures.robot.
Get Thermal Records and Verify | PASS |
Reboot And Check Ambient Temperature Records Are Valid | PASS |

Signed-off-by: Tim Lee <timlee660101@gmail.com>
Change-Id: I6d68001fb04abb989d492cc14198677e4c540132

show more ...

Update test setup to use redfish error log purging

Change-Id: I640fd32ea5597d61e0f46f6a2dcf2c20cf743824
Signed-off-by: George Keishing <gkeishin@in.ibm.com>

redfish: systems: fix unexpected BMC error logs present issue

Symptom:
Auto test got failed from redfish/systems/test_thermal_ambient_temperatures.robot
Error is "Unexpected BMC error log(s) present

redfish: systems: fix unexpected BMC error logs present issue

Symptom:
Auto test got failed from redfish/systems/test_thermal_ambient_temperatures.robot
Error is "Unexpected BMC error log(s) present.: 200 != 404" and result is FAIL
Get Ambient Temperature Records | FAIL |
Reboot And Check Ambient Temperature Records Are Valid | FAIL |

Root cause:
According keyword "Get Thermal Records and Verify" that will get
the thermal records for temperatures and verify whether there is
any error event log. We found that there are lots of log entries
exist at /xyz/openbmc_project/logging/entry/xxx and test item got failed
in keyword "Error Logs Should Not Exist".

Solution:
Before running thermal ambient temperatures test items, we need to
do "Delete All Error Logs" for clear all event logs first in Test Setup.

Tested:
Run robot test redfish/systems/test_thermal_ambient_temperatures.robot.
Get Ambient Temperature Records | PASS |
Reboot And Check Ambient Temperature Records Are Valid | PASS |

Signed-off-by: Tim Lee <timlee660101@gmail.com>
Change-Id: I02003bf1478a26cb716bc6d386d442bf546adf85

show more ...

Fixed cases with hard-coded 'chassis'

Reference to
https://www.dmtf.org/sites/default/files/standards/documents/DSP2046_2020.3.pdf

It should not hard-coded as 'chassis' in /redfish/v1/Chassis/chass

Fixed cases with hard-coded 'chassis'

Reference to
https://www.dmtf.org/sites/default/files/standards/documents/DSP2046_2020.3.pdf

It should not hard-coded as 'chassis' in /redfish/v1/Chassis/chassis.
It could be different name for /redfish/v1/Chassis/{ChassisId}.

Signed-off-by: Tony Lee <tony.lee@quantatw.com>
Change-Id: Ie6e8f37356b88692148bdda7348e715fcac6b4e3

show more ...

Updated threshold arguments due to exceeded character length

Signed-off-by: Marissa Garza <garzam@us.ibm.com>
Change-Id: Iffb94e0732a773b8756c0f0e79295b1ec14c4de3

Updated and added ambient temperature test case

Resolves openbmc/openbmc-test-automation#1965

Signed-off-by: Marissa Garza <garzam@us.ibm.com>
Change-Id: I984485bd2404ebe209eafae2c0823ec812407457

Change Rvalid to Valid

Change all references to keywords like "Rvalid Values" to "Valid Values".
"Rvalid Values" has been deprecated.

Change-Id: I1b418fa69e5753c8bfb9e6a06a8b566359e10cae
Signed-off

Change Rvalid to Valid

Change all references to keywords like "Rvalid Values" to "Valid Values".
"Rvalid Values" has been deprecated.

Change-Id: I1b418fa69e5753c8bfb9e6a06a8b566359e10cae
Signed-off-by: Michael Walsh <micwalsh@us.ibm.com>

show more ...

Remove use of deprecated terse() on print_var

terse is now the default.

Change-Id: If25901eea7fdfa502b012cb995242790e7f6176d
Signed-off-by: Michael Walsh <micwalsh@us.ibm.com>

Update read thermal ambient and sensor temperatures

Signed-off-by: Marissa Garza <garzam@us.ibm.com>
Change-Id: Ib30c1abb04fb9756a3f726226ff2ecb73baabe0b

Read thermal ambient temperatures

Signed-off-by: Marissa Garza <garzam@us.ibm.com>
Change-Id: Ib16e7dbe723bbaebbdbc9e867d0d5d1eaee8e3ca