black: re-format

black and isort are enabled in the openbmc-build-scripts on Python files
to have a consistent formatting. Re-run the formatter on the whole
repository.

Signed-off-by: Patrick Will

black: re-format

black and isort are enabled in the openbmc-build-scripts on Python files
to have a consistent formatting. Re-run the formatter on the whole
repository.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I944f1915ece753f72a3fa654902d445a9749d0f9

show more ...

Revert "black: re-format"

This reverts commit 5731818de0ce446ceaafc7e75ae39da1b69942ae.

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

black: re-format

black and isort are enabled in the openbmc-build-scripts on Python files
to have a consistent formatting. Re-run the formatter on the whole
repository.

Change-Id: If1010ead857d413

black: re-format

black and isort are enabled in the openbmc-build-scripts on Python files
to have a consistent formatting. Re-run the formatter on the whole
repository.

Change-Id: If1010ead857d41364c024bf8145a979a9377d382
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

Add support to run as sudo user on BMC

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

Update python code for python 3.8 or higher compatibility

Changes:
- Update shebang directive to use python3 explicitly.
- remove preexec_fn feature which is deprecated in

Update python code for python 3.8 or higher compatibility

Changes:
- Update shebang directive to use python3 explicitly.
- remove preexec_fn feature which is deprecated in higher
python version 3.8 and above.
- bufsize fix, from documentation bufsize 1 means line buffered
(only usable if universal_newlines=True i.e., in a text mode)
refer: https://docs.python.org/3/library/subprocess.html
- Fix SyntaxWarning: "is" with "=="

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

show more ...

Filter based on REDFISH_SUPPORT_TRANS_STATE for x86 for redfish path

REDFISH_SUPPORT_TRANS_STATE is still a development variable which
will be eventually auto populated when the firmware

Filter based on REDFISH_SUPPORT_TRANS_STATE for x86 for redfish path

REDFISH_SUPPORT_TRANS_STATE is still a development variable which
will be eventually auto populated when the firmware and test is ready
to run is to run in Redfish mode ONLY.

User can force test to run ONLY redfish mode using this varaible
-v REDFISH_SUPPORT_TRANS_STATE:1

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

show more ...

Redfish boot progresss changes

Refer: https://gerrit.openbmc-project.xyz/c/openbmc/bmcweb/+/29344
Change-Id: I34af145431f774d83f4445576bbee8fc30dfccef
Signed-off-by: George Keishing

Redfish boot progresss changes

Refer: https://gerrit.openbmc-project.xyz/c/openbmc/bmcweb/+/29344
Change-Id: I34af145431f774d83f4445576bbee8fc30dfccef
Signed-off-by: George Keishing <gkeishin@in.ibm.com>

show more ...

Add retry post state check keyword timedout

The robot "Redfish Get States" keyword timed out after 2 minutes.
This is to re-try one more time to improve boot test success rate.

Add retry post state check keyword timedout

The robot "Redfish Get States" keyword timed out after 2 minutes.
This is to re-try one more time to improve boot test success rate.

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

show more ...

Add a possibility to set PLATFORM_ARCH_TYPE with an environment variable

Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
Change-Id: I364335d4cec9d258354bd3f52e75f452886dc4db

Fix conflicting "CurrentPowerState" chassis state from chassis_system0 on x86

Changes:
Boot table look up "CurrentPowerState" from the chassis
URL "/xyz/openbmc_project/state

Fix conflicting "CurrentPowerState" chassis state from chassis_system0 on x86

Changes:
Boot table look up "CurrentPowerState" from the chassis
URL "/xyz/openbmc_project/state/chassis0".
On x86 there is the 'chassis_system0' URL that has the same property
that is needed to be skiped in a state enumeration logic to prevent
unnecessary state override.

"/xyz/openbmc_project/state/chassis0": {
"CurrentPowerState": "On",

"/xyz/openbmc_project/state/chassis_sytem0": {
"CurrentPowerState": "Off",

Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
Change-Id: I772fce4a8dbbd079829e4572d60b39ebf3727c07

show more ...

Add x86 infrastruction for boot test

Changes:
- Add platform type user driven option "x86". by default it will
choose platform "power"
- Add filter to removed x86 non sup

Add x86 infrastruction for boot test

Changes:
- Add platform type user driven option "x86". by default it will
choose platform "power"
- Add filter to removed x86 non supported states from state.py
- Remove "operating_systems" entry from data/boot_table_x86.json

How To test:
robot -v OPENBMC_HOST:xx.xx.xx.xx -v PLATFORM_ARCH_TYPE:x86 <test suite>

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

show more ...

Add SSH time_out enforce when called during BMC reset

This is to prevent SSH to BMC getting hang if the execution on
BMC to get uptime stalled during reset.

Change-Id: Ic6a6217e

Add SSH time_out enforce when called during BMC reset

This is to prevent SSH to BMC getting hang if the execution on
BMC to get uptime stalled during reset.

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

show more ...

New x86 boot table and required changes

Changes:
- New boot table for x86 OpenBMC build which doesn't support
BootProgress.
- Initialize state.py boot_progress to m

New x86 boot table and required changes

Changes:
- New boot table for x86 OpenBMC build which doesn't support
BootProgress.
- Initialize state.py boot_progress to make it optional if it
fails in state URI enumeration.

Note: We don't have x86 setup to test the changes.

To test use bellow option in robot CLI

-v BOOT_TABLE_PATH:data/boot_table_x86.json

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

show more ...

Support Wait State keyword inside plugin.

Signed-off-by: Michael Shepos <shepos@us.ibm.com>
Change-Id: If6120c480efdc65ab031fbdb0570b0b19860a768

Fix conflicting "CurrentHostState" host state from hypervisor

Changes:
Boot table look up "CurrentHostState" but from the host
URL "/xyz/openbmc_project/state/host0". Skippin

Fix conflicting "CurrentHostState" host state from hypervisor

Changes:
Boot table look up "CurrentHostState" but from the host
URL "/xyz/openbmc_project/state/host0". Skipping the
hypervisor state from the state enumeration logic for
checking the boot state.

"/xyz/openbmc_project/state/host0": {
"CurrentHostState": "xyz.openbmc_project.State.Host.HostState.Running",

"/xyz/openbmc_project/state/hypervisor0": {
"CurrentHostState": "xyz.openbmc_project.State.Host.HostState.TransitioningToRunning",

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

show more ...

Changes to read the redfish boot progress state

Signed-off-by: Michael Shepos <shepos@us.ibm.com>
Change-Id: I17557c1dbff5e9b26bb0d305db02758cbfff6272

Changes to support Redfish boot progress

Signed-off-by: Michael Shepos <shepos@us.ibm.com>
Change-Id: Icfe34896b5801f13a344b6a3983696796dc476cb

Increase wait_for_comm_cycle to 12 mins

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

Increase read properties timeout in state.py

- Timeouts have occured in simulation firmware testing. Increase
timeout from 10 to 30 seconds to fix the problem.

Signed

Increase read properties timeout in state.py

- Timeouts have occured in simulation firmware testing. Increase
timeout from 10 to 30 seconds to fix the problem.

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

show more ...

wait_for_comm_cycle to wait for uptime

- Modify wait_for_comm_cycle() to wait for uptime to be less than
elapsed_boot_time. It will no longer insist that packet_loss reach
100 a

wait_for_comm_cycle to wait for uptime

- Modify wait_for_comm_cycle() to wait for uptime to be less than
elapsed_boot_time. It will no longer insist that packet_loss reach
100 and that it then reach 0.
- Modify compare_states() function to evaluate python expressions in
addition to processing regular expressions.
- Modify get_state to get elapsed_boot_time
- Miscellaneous cleanup:
- valid_value, etc. no longer needs to be fed var_name.
- "Description of arguments" syntax was not adhering to standard.
- Changed anchor_state() and strip_anchor_state() to stop reading
values from state dictionary. The first goal was to stop using
the name match_state_value since the function need not presume that
the dictionary is a match dictionary.

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

show more ...

Adjust to deprecated validation functions

- The "svalid" functions (e.g. svalid_value) have been deprecated. The
new method is to use the corresponding "valid" function (e.g.
va

Adjust to deprecated validation functions

- The "svalid" functions (e.g. svalid_value) have been deprecated. The
new method is to use the corresponding "valid" function (e.g.
valid_value).

- Also, valid_range now takes lower and upper args (vs. the old
"<lower>..<upper>" string).

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

show more ...

get_state process uptime in test_mode

Made changes to get_state() function to process uptime requests even when
test_mode is set to 1.

Change-Id: I5e2309996ae84ad31b736529adb8e4

get_state process uptime in test_mode

Made changes to get_state() function to process uptime requests even when
test_mode is set to 1.

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

show more ...

DMTF tool power on operation

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

Remove use of newly deprecated rprint_issuing_keyword

The gen_print.py print_issuing function will now handle robot commands that
are formed as a list.

Change-Id: I8f3f22d55afbb

Remove use of newly deprecated rprint_issuing_keyword

The gen_print.py print_issuing function will now handle robot commands that
are formed as a list.

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

show more ...

Support early exit from wait_until_keyword_succeeds in wait_state

Currently, wait_until_keyword_succeeds() doesn't provide a means for a
programmer to exit early for a special condition.

Support early exit from wait_until_keyword_succeeds in wait_state

Currently, wait_until_keyword_succeeds() doesn't provide a means for a
programmer to exit early for a special condition.

This commit contains a solution whereby a signal handler can be invoked
to set a global wait_early_exit_message to indicate to check_state and
wait_state that an early exit should be made and it should count as a
failure.

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

show more ...