Managers ID : Introduced variable to declare Managers ID in resource.robot and updated all the scripts in openbmc-test-automation

Changes:
- Introduced ${MANAGER_ID} variable in resource.robot

Managers ID : Introduced variable to declare Managers ID in resource.robot and updated all the scripts in openbmc-test-automation

Changes:
- Introduced ${MANAGER_ID} variable in resource.robot
- Replaced all the Managers/bmc to Managers/${MANAGER_ID} across all the scripts
- Also in data/variables.py updated chassis id for the few redfish uri.

Tested:
- Ran All The Scripts Successfully

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

show more ...

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

Fix regex expression to find list of end point objects

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

lib: skip Journal in redfish enumerate

Symptom:
"Verify Redfish Unresponsive URL paths" test item got failed.
ValueError: The HTTP status code was not valid:
status:

lib: skip Journal in redfish enumerate

Symptom:
"Verify Redfish Unresponsive URL paths" test item got failed.
ValueError: The HTTP status code was not valid:
status: 400
valid_status_codes:
[0]: 200
[1]: 404
[2]: 405
[3]: 500

Root cause:
When doing Redfish Enumerate Request into /redfish/v1/Managers/bmc/LogServices/Journal,
The response status code return 400 that is not in valid_status_codes list.

Seems journal entries will be overwrited when entries full.
Thus, when enumerating the overwrited log entries will got failed.
Currently, "OverWritePolicy" is "WrapsWhenFull" in LogService.json schema.

Solution:
Skip Journal in redfish enumerate request.

Tested:
Run all robot test cases.

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

show more ...

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

Skip Registries and PostCode in redfish enumerate

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

Revert "Included Redfish sessions data"

Hit FFDC data collection issues.

This reverts commit 6756d443c22237ebd4bd778f4d41cf235bf0166f.

Reason for revert: <INSERT REASONING

Revert "Included Redfish sessions data"

Hit FFDC data collection issues.

This reverts commit 6756d443c22237ebd4bd778f4d41cf235bf0166f.

Reason for revert: <INSERT REASONING HERE>

Change-Id: Ia17d3db6840ab3160e18971764e5a4e75bdbb3c7

show more ...

Included Redfish sessions data

Change-Id: I68793227d3a1f65a0441ffbdc8b352e9a2cd97a4
Signed-off-by: Anusha Dathatri <adathatr@in.ibm.com>

Fixed TypeError for member URIs

Signed-off-by: Anusha Dathatri <adathatr@in.ibm.com>
Change-Id: Iae35f0d0a0963d719394aa2a6556b1f97273c2f3

Add support to collect OEM redfish path in FFDC

Changes:
- Added new keyword to collect OEM redfish properties.
- Added new variable OEM_REDFISH_PATH

Tested:

Add support to collect OEM redfish path in FFDC

Changes:
- Added new keyword to collect OEM redfish properties.
- Added new variable OEM_REDFISH_PATH

Tested:
robot -v OPENBMC_HOST:xx.xx.xx.xx -v OEM_REDFISH_PATH:/ibm/v1 tools/myffdc.robot

User can key in their own OEM path in the command line as above.

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

show more ...

Fix to get correct target action URI

Signed-off-by: Anusha Dathatri <adathatr@in.ibm.com>
Change-Id: I4ba4c1c20c07401f94eb9a41e62c5cd06a422b41

Add mTLS option for redfish test

When the BMC only enables mTLS auth type at build time. Robot test
should be test normally. Adding "MTLS_ENABLED" environment variable as
an option a

Add mTLS option for redfish test

When the BMC only enables mTLS auth type at build time. Robot test
should be test normally. Adding "MTLS_ENABLED" environment variable as
an option and modifications to achieve it.

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

show more ...

Fix typo spelling in the test documentation

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

Filter out JsonSchemas, SessionService and # from listed URI

The changes is made to avoid looking up schemas, session objects
and path like /redfish/v1/Managers/bmc#/Oem to avoid GET req

Filter out JsonSchemas, SessionService and # from listed URI

The changes is made to avoid looking up schemas, session objects
and path like /redfish/v1/Managers/bmc#/Oem to avoid GET request
failure or spent time iterating it.

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

show more ...

Update IPMI test case

Changes:
- Add code to dynamically load the PowerControl redfish URI.
- Adjust debug log and logging request in library.

Resolves openbmc/openb

Update IPMI test case

Changes:
- Add code to dynamically load the PowerControl redfish URI.
- Adjust debug log and logging request in library.

Resolves openbmc/openbmc-test-automation#2091

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

show more ...

Add support to dynamically find valid path in redfish using attribute

Changes:
- Function to walk through a given base path to find the path which
holds the attribute.

Add support to dynamically find valid path in redfish using attribute

Changes:
- Function to walk through a given base path to find the path which
holds the attribute.

This changes enabled the test to just provide the base path redfish URI
and the attribute a parent or child holds and returns back the list of
path(s).

Sample Test:
redfish_utils.Get Members URI /redfish/v1/Chassis/ PowerSupplies
redfish_utils.Get Members URI /redfish/v1/Chassis/ PowerControl

would return to the caller in the test as:
['/redfish/v1/Chassis/chassis/Power']
and
['/redfish/v1/Chassis/chassis/Power',
'/redfish/v1/Chassis/motherboard/Power']

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

show more ...

Add support to find a list of redfish URL ending with endpoints

Chamges:
- Function to return list of entries ending with /endpoint
For end point "Power":

Add support to find a list of redfish URL ending with endpoints

Chamges:
- Function to return list of entries ending with /endpoint
For end point "Power":
['/redfish/v1/Chassis/chassis/Power',
'/redfish/v1/Chassis/motherboard/Power']

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

show more ...

REST and Redfish supported driver

Changes:
- Added REDFISH_REST_SUPPORTED global var as part of the
lib/bmc_redfish_utils.py init function.
- Update boot test to se

REST and Redfish supported driver

Changes:
- Added REDFISH_REST_SUPPORTED global var as part of the
lib/bmc_redfish_utils.py init function.
- Update boot test to select supported BMC REST or Reddfish
fucntions to set.

There are driver still out there in community which supports both
legacy REST (/xyz/openbmc_project/) and new redfish (/redfish/v1/).
This new logic is to derieve early in the program and use in the
functional and boot test path.

Tested:
- Disable REST and test the change.
- Test the change in current latest level of driver.

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

show more ...

Added test to verify unresponsive paths in redfish

Signed-off-by: Anusha Dathatri <adathatr@in.ibm.com>
Change-Id: I789c07398e04d162b0b9a92d3f74a19a50139a13

Modified enumerate_request method to print broken URI resources on demand

Signed-off-by: Anusha Dathatri <adathatr@in.ibm.com>
Change-Id: I2cb044d011696e6c6d548c70f55b97ebbde6c3a7

Exclude session information from Redfish enumeration

Change-Id: I7dffc368916e01103c0b4fdc407eda9d678ad50d
Signed-off-by: Anusha Dathatri <adathatr@in.ibm.com>

Verify repeated failed logins cause lockout

Verify that exceeding the failed login threshold by repeatedly trying to login
with an invalid password causes the user account to be locked.

Verify repeated failed logins cause lockout

Verify that exceeding the failed login threshold by repeatedly trying to login
with an invalid password causes the user account to be locked.

Change-Id: I1e8f5990a3cd215004494f04b9cf4819607c7922
Signed-off-by: Sandhya Somashekar <sansomas@in.ibm.com>

show more ...

enumerate_request to support return_json arg

Added code to enumerate_request method to support a new return_json argument.
This new argument indicates whether the result should be return

enumerate_request to support return_json arg

Added code to enumerate_request method to support a new return_json argument.
This new argument indicates whether the result should be returned as a json
string or as a dictionary.

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

show more ...