Added new script for IPMI POH Counter Support

POH (Power-On Hours) counter has the incremental count of power ON hours of the system.

Request and Response data defined under data/ipmi_raw_cmd_table

Added new script for IPMI POH Counter Support

POH (Power-On Hours) counter has the incremental count of power ON hours of the system.

Request and Response data defined under data/ipmi_raw_cmd_table.py

Testcases added -
- Get POH Counter Command Via IPMI
- Verify Get POH Counter With Invalid Data Request Via IPMI
- Verify POH Counter Reading With Wait Time
- Verify POH Counter Reading With Host Power Off
- Verify POH Counter Reading With Host Power On

Script compares Minutes per count and Counter reading for the above scenarios.

Minutes per count usually 60 minutes.

Wait Time given - 1 hour in host powered on state, 1 hour 30 minutes when Host power OFF, 1 hour after Host Power ON

Comparison between Initial POH Counter reading and reading after wait time / Power operation.

Tested: Run robot ipmi/test_ipmi_poh_counter.robot

Signed-off-by: chithrag <chithrag@ami.com>
Change-Id: I0137f42cc4af215abfdeef6ad0d9004e9dd2d2dc

show more ...

Created New script for IPMI Set/Get SEL Time

Pre-requisite Condition : Client Machine and BMC should be in Same TimeZone (example : UST).
Mismatch of Timezones result in miscalculation of SEL Time d

Created New script for IPMI Set/Get SEL Time

Pre-requisite Condition : Client Machine and BMC should be in Same TimeZone (example : UST).
Mismatch of Timezones result in miscalculation of SEL Time due to different Timezone in Client Machine.

IPMI Raw command variables are defined under ../data/ipmi_raw_cmd_table.py
Python basic functionalities are defined under lib/ipmi_client.robot, lib/ipmi_utils.py and lib/utils.py files.

Test the Set/Get SEL Time functionality and compare the result against BMC Native command (date).

Set the Time Sync Mode from NTP to Manual to Set SEL Time.
Time Sync Mode change performed via REDFISH URI.
Performs the change in Time Sync Mode with Test Setup and Teardown Execution with default NETWORK_TIMEOUT provided under ../lib/resource.robot

NETWORK_RESTART_TIME added for Set SEL Time and Add SEL Entry as the corresponding command takes approx 5 seconds for the operation to reflect.

Current SEL time identified via BMC Native command (date) and perform SEL Time operations.

Script Verifies SEL Time for various scenarios such as,
- Get current time from BMC and add future year and compare against BMC native command (date),
- Gets BMC Current Time and Adds 15 minutes and compare against BMC native command (date),
- Gets BMC Current Time and subtracts 1 day and compare against BMC native command (date),
- Add SEL Entry for all the above scenarios and compare against BMC native command (date).

Tested: Run robot ipmi/test_ipmi_sel_time.robot

Signed-off-by: chithrag <chithrag@ami.com>
Change-Id: Ied129b96e38351d59780326c71f875dc50ce33f5

show more ...

Moved a keyword from certificate suite to utils

Change-Id: I51935be6e1cf16cbffc75f7ccb3d05ceaf692a59
Signed-off-by: Rahul Maheshwari <rahulmaheshwari@in.ibm.com>

Update redfish delete session logic

Changes:
- Added keyword to get session with client id populated.
- Added logic to remove sessions with client id.

Change-Id: Ifee7867315144f786764ff

Update redfish delete session logic

Changes:
- Added keyword to get session with client id populated.
- Added logic to remove sessions with client id.

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

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 ...

Wait on the BootProgress state change

Changes:
- New keyword for Boot Progress change
- Remove the explicit sleep
- Wait on BootProgress changed from None

Change-Id: I336557de61c3f97

Wait on the BootProgress state change

Changes:
- New keyword for Boot Progress change
- Remove the explicit sleep
- Wait on BootProgress changed from None

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

show more ...

lib: state_manager: support x86 platform get OperatingSystemState attribute

Symptom:
When host CPU based on the x86 architecture, the attribute OperatingSystemState
cannot get correctly then cause t

lib: state_manager: support x86 platform get OperatingSystemState attribute

Symptom:
When host CPU based on the x86 architecture, the attribute OperatingSystemState
cannot get correctly then cause test case got failed.

Root cause:
Based on x86 architecture, using x86-power-control to manager host power
operation that add OS state interface at "/xyz/openbmc_project/state/os"
this object path. However, "Get Host State Attribute" keyword in
state_manager.robot always using "/xyz/openbmc_project/state/host0" by default.
Thus, cause this OperatingSystemState attribute cannot be found.

x86-power-control:
// OS State Interface
osIface = osServer.add_interface(
"/xyz/openbmc_project/state/os",
"xyz.openbmc_project.State.OperatingSystem.Status");

openbmc-test-automation/data/variables.py:
HOST_STATE_URI = /xyz/openbmc_project/state/host0/

Solution:
Add support x86 platform to get OperatingSystemState attribute.

Tested:
Run keyword "Is OS Booted" with -v PLATFORM_ARCH_TYPE:x86
${HOST_STATE_URI} = /xyz/openbmc_project/state/os/

Run keyword "Is OS Booted" with -v PLATFORM_ARCH_TYPE:power
${HOST_STATE_URI} = /xyz/openbmc_project/state/host0/

Related commit:
Suggest Automation test should be adjusted to match the design in this case.
https://gerrit.openbmc-project.xyz/c/openbmc/smbios-mdr/+/51691

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

show more ...

Host inid reset boot table and wait changes

Changes:
- bmc_reboot:1 directive to check for wait and change state
- Remove ping check from utils code for host reset

Change-Id: Iab24927570e9a

Host inid reset boot table and wait changes

Changes:
- bmc_reboot:1 directive to check for wait and change state
- Remove ping check from utils code for host reset

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

show more ...

sudo user option update for BMC execute command

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

Enforce BMC redfish state for BMC reset

Changes:
- Added directive in boot table for BMC state check
on BMC reset.
- remove 3 minute wait for ping and let state check
take over i

Enforce BMC redfish state for BMC reset

Changes:
- Added directive in boot table for BMC state check
on BMC reset.
- remove 3 minute wait for ping and let state check
take over it.

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

show more ...

Fix robot directive Keywords missing header

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

Update robot FFDC CLI command

JSON files no longer available under /var/lib/bmcweb/

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

Collect all dumps available on BMC

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

Feature: IPMI user

Updated below test cases from test_ipmi_user.robot script.

- Verify Administrator And No Access Privilege For Different Channels

- Verify Operator And User

Feature: IPMI user

Updated below test cases from test_ipmi_user.robot script.

- Verify Administrator And No Access Privilege For Different Channels

- Verify Operator And User Privilege For Different Channels

Removed hardcoded channel number.

Added keyword to get all available channel number.

Added keyword to get number of active channel number for ethernet interface.

Added keyword to skip different channel test case based active channel number count
and added this keyword in test setup for different channel test cases.

Signed-off-by: ganesanb <ganesanb@ami.com>
Change-Id: I9ac6e7cef973fbc3b692d3ce4c1fd52dd2918440

show more ...

Dump URI listing check if redfish

This is not needed in case of redfish as it's get captured
as part of the /redfish/v1/ enumeration.

Change-Id: Id6f537384fd60d5d31c72f1862460a72d3000f37
Signed-off

Dump URI listing check if redfish

This is not needed in case of redfish as it's get captured
as part of the /redfish/v1/ enumeration.

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

show more ...

FFDC pre and post files cleanup

Changes:
- Cleanup files from BMC /tmp/ collected as part of FFDC.

Change-Id: I2e852aefc6b8826c0e05be9cbf4f4f8d5c37dea2
Signed-off-by: George Keishing <gkeishin@

FFDC pre and post files cleanup

Changes:
- Cleanup files from BMC /tmp/ collected as part of FFDC.

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

show more ...

Add BIOS attribute patch setting keyword

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

Update Redfish-Reference-Checker and Redfish-Service-Validator suites

Changes:
- lib/dmtf_tools_utils.robot to return the RC
- Update test code to handle rc returnning

Resolves openbmc/ope

Update Redfish-Reference-Checker and Redfish-Service-Validator suites

Changes:
- lib/dmtf_tools_utils.robot to return the RC
- Update test code to handle rc returnning

Resolves openbmc/openbmc-test-automation#2178
Resolves openbmc/openbmc-test-automation#2179

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

show more ...

Added new script to verify IPMI cmd via lanplus interface.

And also added a case to execute lanplus multiple times.

Created new resource file with keyword which contains Few ipmi commands
to execut

Added new script to verify IPMI cmd via lanplus interface.

And also added a case to execute lanplus multiple times.

Created new resource file with keyword which contains Few ipmi commands
to execute over lanplus interface.

We can add more ipmi commands in future.

Tested: Run robot ipmi/test_ipmi_lanplus.robot

Signed-off-by: Nagarjun B <nagarjunb@ami.com>
Change-Id: Ic8f559e47b908a3b12af598a6e2b0c81bf947a37

show more ...

Increase OCC_WAIT_TIMEOUT to 4 minutes

Signed-off-by: David Shaw <dlshaw@us.ibm.com>
Change-Id: I6dbe6248fcadaada7365978cbdc0b42450bf8c7e

Added new script to verify the IPMI KCS cmd.

created a testcase to run the IPMI KCS cmd overnight.

Created Resource file for KCS interface commands which
contains few of the IPMI commands to verify

Added new script to verify the IPMI KCS cmd.

created a testcase to run the IPMI KCS cmd overnight.

Created Resource file for KCS interface commands which
contains few of the IPMI commands to verify Kcs interface.

We can add other ipmi cmds in future.

Added IPMI raw commands in ipmi_raw_cmd_table.py

Added "Is BMC Operational" keyword instead of Sleep time,
to verify BMC is enabled via redfish.

Tested: Run robot /ipmi/test_ipmi_kcs.robot

Signed-off-by: Nagarjun B <nagarjunb@ami.com>
Change-Id: I59f1515792ab99fb5d1f55566ea3c0fcfcc2942b

show more ...

Get BIOS attribute keyword

Returns BIOS attribute dictionary.

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

Provision image upload timeout

Changes:
- Add IMAGE_UPLOAD_WAIT_TIMEOUT and default to 4 minutes

Usage:
User can specify -v IMAGE_UPLOAD_WAIT_TIMEOUT:10

Change-Id: I53e52213bf5215b714aaf744

Provision image upload timeout

Changes:
- Add IMAGE_UPLOAD_WAIT_TIMEOUT and default to 4 minutes

Usage:
User can specify -v IMAGE_UPLOAD_WAIT_TIMEOUT:10

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

show more ...

FFDC to handle root and sudo user

Added bash -c; just to fake load the command bash so that
it doesnt hang in bash and continue execution shell command

Change-Id: I5254a2ed83e4ca8b4fef0edf305564150

FFDC to handle root and sudo user

Added bash -c; just to fake load the command bash so that
it doesnt hang in bash and continue execution shell command

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

show more ...

Add skip error option for journald log checking

Usage:
Single string pattern option usage:
-v SKIP_ERROR:'core-dump'

Multiple string pattern "\|" separated option usage:
-v SKIP_ERR

Add skip error option for journald log checking

Usage:
Single string pattern option usage:
-v SKIP_ERROR:'core-dump'

Multiple string pattern "\|" separated option usage:
-v SKIP_ERROR:'core-dump/Failed to start OpenPOWER OCC'

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

show more ...