Robot Framework 7.0 changes

Update robot code syntax overall in the repo

Changes:
- Update the requirement.txt package version for robot
- The '[Return]' setting is deprecated.
Use the

Robot Framework 7.0 changes

Update robot code syntax overall in the repo

Changes:
- Update the requirement.txt package version for robot
- The '[Return]' setting is deprecated.
Use the 'RETURN' statement instead.
- Singular section headers like '*** Test Case ***'
are deprecated. Use plural format like
'*** Test Cases ***' instead.
- Singular section headers like '*** Variable ***'
are deprecated

Tested:
- Ran from sandbox with the version
$ robot --version
Robot Framework 7.0 (Python 3.10.12 on linux)

and also tested on latest python version

Robot Framework 7.0 (Python 3.11.9 on linux)

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

show more ...

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

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

Update Clean up script to handle REST and redfish

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

Redfish v1 boot state mapping update for boot table JSON

Changes:
- Update boot table to redfish mapping state
- Added GET current active session info for reference

Change-Id: I3f599194

Redfish v1 boot state mapping update for boot table JSON

Changes:
- Update boot table to redfish mapping state
- Added GET current active session info for reference

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

show more ...

Keyword to read the redfish boot progress state

Changes:
- New keyword to GET and return the boot progress state.
- Add new keyword to return all the system states.

boot state example:

Keyword to read the redfish boot progress state

Changes:
- New keyword to GET and return the boot progress state.
- Add new keyword to return all the system states.

boot state example:
{
'bmc': 'Enabled',
'chassis': 'Off',
'host': 'Disabled',
'boot_progress': 'None'
}

{
'bmc': 'Enabled',
'chassis': 'On',
'host': 'Enabled',
'boot_progress': 'SystemHardwareInitializationComplete'
}

{
'bmc': 'Enabled',
'chassis': 'On',
'host': 'Enabled',
'boot_progress': 'OSRunning'
}

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

show more ...

Support new redfish BMC and host state keywords

Changes:
- Keyword to return BMC current state.
- Keyword to return host states (power and boot progress state).

Change-Id: I2efc47bf4b56

Support new redfish BMC and host state keywords

Changes:
- Keyword to return BMC current state.
- Keyword to return host states (power and boot progress state).

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

show more ...

Fix boot sensor test

Changes:
- Added "wait" option for "Initiate Host Reboot" keyword.
- Update system state in test template.

Change-Id: Ia7c714694a064b67615137f300e344bdbdcf7777
Signed-o

Fix boot sensor test

Changes:
- Added "wait" option for "Initiate Host Reboot" keyword.
- Update system state in test template.

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

show more ...

Enforced robotframework standards.

- The standards are enforced using the rflint static anylysis tool.
- More standards will be added in future commits but this is a base
starting point.

Change-I

Enforced robotframework standards.

- The standards are enforced using the rflint static anylysis tool.
- More standards will be added in future commits but this is a base
starting point.

Change-Id: I7dd8a2b0b42a51327b748cb730609d7ee2538de3
Signed-off-by: Joy Onyerikwu <onyekachukwu.joy.onyerikwu@ibm.com>

show more ...

Cleanup deprecated org state code

Resolves openbmc/openbmc-test-automation#986

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

Add wait argument to 'Initiate BMC Reboot'.

This support is needed for OBMC Boot Test to use this function.

Change-Id: Idb907e072343602fc5467acebd97511bab02b0e8
Signed-off-by: Michael Walsh <micwal

Add wait argument to 'Initiate BMC Reboot'.

This support is needed for OBMC Boot Test to use this function.

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

show more ...

Added support for new 'REST Hard Power Off' test.

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

Removing org poweron logic from state manager

Resolves openbmc/openbmc-test-automation#932

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

Hard power off if the soft power off failed

Resolves openbmc/openbmc-test-automation#821

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

OS boot progress and status changes

Resolves openbmc/openbmc-test-automation#842

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

Disable BootProgress sensor test on power off

The firmware does not reset this sensor on power
off's currently.

Change-Id: I12d680a81b68ad622a0f31c90f97eeb0e442b74c
Signed-off-by: Andrew Geissler <

Disable BootProgress sensor test on power off

The firmware does not reset this sensor on power
off's currently.

Change-Id: I12d680a81b68ad622a0f31c90f97eeb0e442b74c
Signed-off-by: Andrew Geissler <andrewg@us.ibm.com>

show more ...

Add chassis and "BootProgress" state check for power on and off

Resolves openbmc/openbmc-test-automation#802

Change-Id: I6ab25ff7128d05378077ed34eb940e373b695648
Signed-off-by: George Keishing <gk

Add chassis and "BootProgress" state check for power on and off

Resolves openbmc/openbmc-test-automation#802

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

show more ...

Fix Poweroff wait time and remove wait from test

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

Fix for connection reset by peer error during BMC reboot.

Connection reset by peer error means an immediate dropping
of the connection. This error is expected and should be
ignored.

Resolves openbm

Fix for connection reset by peer error during BMC reboot.

Connection reset by peer error means an immediate dropping
of the connection. This error is expected and should be
ignored.

Resolves openbmc/openbmc-test-automation#509

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

show more ...

Changed OBMC_STATES_VERSION default from 0 to 1.

Also corrected incorrect comment.

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

Add wait option to keywords from lib/state_manager.robot.

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

Fix for "Is Host Quiesced" keyword not found.

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

Enable host quiesced check at the end of test case.

This change includes:
- Collect FFDC and fails test case if host reaches
quiesced at the end of test case run.
- Also recovers host from q

Enable host quiesced check at the end of test case.

This change includes:
- Collect FFDC and fails test case if host reaches
quiesced at the end of test case run.
- Also recovers host from quiesce state.

Resolves openbmc/openbmc-test-automation#322

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

show more ...

Added keyword to initiate and check host reboot.

This change includes:
- Added keyword to initiate and check host reboot
using new xyz interface.
- Seperated keyword for host and chassis sta

Added keyword to initiate and check host reboot.

This change includes:
- Added keyword to initiate and check host reboot
using new xyz interface.
- Seperated keyword for host and chassis state check.
This is required as host and chassis state does not
always match e.g. host reboot.
- Added a keyword to check host's transition state.

Resolves openbmc/openbmc-test-automation#350

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

show more ...

Corrected the logic to set version variable.

Earlier logic was to check org interface to decide. As older
org interfaces are always present, automation code was always
taking org interface.

Now wit

Corrected the logic to set version variable.

Earlier logic was to check org interface to decide. As older
org interfaces are always present, automation code was always
taking org interface.

Now with changes we take decision based on xyz interface. So
if xyz interface available, we set version variable to 1 else 0.

Also corrected setting OBMC_STATES_VERSION to 0 for old
interface and 1 for new xyz interface.

Resolves openbmc/openbmc-test-automation#387

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

show more ...