Modified Get Partial SDR raw command.

We have to get the SDR record data from offset 0x01 for partial get SDR,
previously code was having 0x00 offset.

Tested: Run robot ipmi/test_ipmi_inband_sdr.ro

Modified Get Partial SDR raw command.

We have to get the SDR record data from offset 0x01 for partial get SDR,
previously code was having 0x00 offset.

Tested: Run robot ipmi/test_ipmi_inband_sdr.robot

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

show more ...

Modified IPMI raw command for partial get SDR.

Added code to get SDR record ID dynamically in Suite Setup.
Modified IPMI raw commad for Get partial SDR in data/ipmi_raw_cmd_table.py
and in script fi

Modified IPMI raw command for partial get SDR.

Added code to get SDR record ID dynamically in Suite Setup.
Modified IPMI raw commad for Get partial SDR in data/ipmi_raw_cmd_table.py
and in script file to form IPMI raw command.

Added sleep time of 10s after BMC reboot to populate SDR.

Tested: Run robot ipmi/test_ipmi_sdr.robot

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

show more ...

Added testcases for get channel payload support/version IPMI command.

Testcases added:
1) Verify Get Channel Payload Version
Verify get channel payload version IPMI command for active chann

Added testcases for get channel payload support/version IPMI command.

Testcases added:
1) Verify Get Channel Payload Version
Verify get channel payload version IPMI command for active channels.

2) Verify Get Channel Payload Version For Invalid Channel
Verify get channel payload version IPMI command for all
inactive/invalid channels.
3) Verify Get Channel Payload Support
Verify get channel payload support for all payload types in
active channels.
4) Verify Get Channel Payload Support For Invalid Channel
Verify get channel payload support for all payload types in
inactive/invalid channels.

Added IPMI raw in ipmi_raw_cmd_table.py python variable file.

Tested: Run robot ipmi/test_ipmi_payload.robot

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

show more ...

Fix Tag and typo in documentation

Changes:
- + gui/gui_test/operations_menu/test_host_console_sub_menu.robot
W: 41, 0: Too many trailing blank lines (TrailingBlankLines)
- Tag mismatch

Fix Tag and typo in documentation

Changes:
- + gui/gui_test/operations_menu/test_host_console_sub_menu.robot
W: 41, 0: Too many trailing blank lines (TrailingBlankLines)
- Tag mismatch

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

show more ...

Update LED IndicatorLED location lookup from URI(s)

Example:
The logic can fetch one or more from Systems
/redfish/v1/Systems/system
/redfish/v1/Systems/hypervisor
but all of them wont have the Indi

Update LED IndicatorLED location lookup from URI(s)

Example:
The logic can fetch one or more from Systems
/redfish/v1/Systems/system
/redfish/v1/Systems/hypervisor
but all of them wont have the IndicatorLED implemented.

The code looks up which URI has the indicator and check against it.

Tested:
robot -v OPENBMC_HOST:xx.xx.xx.xx --include Verify_Chassis_Identify_via_IPMI
ipmi/test_ipmi_general.robot

------------------------------------------------------------------------------
Test Ipmi General :: Module to test IPMI asset tag functionality. | PASS |
1 test, 1 passed, 0 failed
==============================================================================

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

show more ...

Add IPMI_POWEROFF_WAIT_TIMEOUT varaible for user input

By default 3 minutes, however on a booted system with partition(s),
a soft power off can take longer. This give flexibility to override
the def

Add IPMI_POWEROFF_WAIT_TIMEOUT varaible for user input

By default 3 minutes, however on a booted system with partition(s),
a soft power off can take longer. This give flexibility to override
the default and test can wait on user input.

Tested:
robot -v OPENBMC_HOST:xx.xx.xx.xx -v IPMI_POWEROFF_WAIT_TIMEOUT:10
--include Verify_Soft_Shutdown ipmi/test_ipmi_chassis.robot

Test Ipmi Chassis :: Module to test IPMI chassis functionality. | PASS |
1 test, 1 passed, 0 failed

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

show more ...

Fix IPMI test to handle where a system has root disabled

By default IPMI_USERNAME is root, if user decides not to use
root or a system where root is disable, this code will now handle
it to work acc

Fix IPMI test to handle where a system has root disabled

By default IPMI_USERNAME is root, if user decides not to use
root or a system where root is disable, this code will now handle
it to work accordingly.

Tested: robot -v OPENBMC_HOST:xx.xx.xx.xx --include Verify_IPMI_User_Deletion ipmi/test_ipmi_user.robot

- With root disabled and use admin user
- With root enabled.

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

show more ...

Corrected typo in the test filename test_ipmi_network_verificaton

- There is a typo in the filename, should be test_ipmi_network_verification.robot

Signed-off-by: Jorge Cisneros <jcisneros3@lenov

Corrected typo in the test filename test_ipmi_network_verificaton

- There is a typo in the filename, should be test_ipmi_network_verification.robot

Signed-off-by: Jorge Cisneros <jcisneros3@lenovo.com>
Change-Id: Id5bb11e4eb774a30c51693922ea3c075ce8f54d9

show more ...

Added new cases for IPMI Set/Get User access payload.

Testcases Added:
1) Verify Set User Access Payload For Invalid User
2) Verify Set User Access Payload for Invalid Channel Number
3) Verify

Added new cases for IPMI Set/Get User access payload.

Testcases Added:
1) Verify Set User Access Payload For Invalid User
2) Verify Set User Access Payload for Invalid Channel Number
3) Verify Get User Access Payload For Invalid User
4) verify Get User Access Payload for Invalid Channel Number
Above all cases try to set/get user access payload for invalid channel/user
and expects error response.

Added Suite Setup to get valid and invalid channels from channel_config.json file

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

show more ...

Add fail_on_err=0 To Run_IPMI_Standard_Command When Expecting Error

Add fail_on_err=0 To Run_IPMI_Standard_Command in line 58
in test_get_bios_post_code.robot in order to prevent test case
failure w

Add fail_on_err=0 To Run_IPMI_Standard_Command When Expecting Error

Add fail_on_err=0 To Run_IPMI_Standard_Command in line 58
in test_get_bios_post_code.robot in order to prevent test case
failure when expecting an error response code

Signed-off-by: Lee Tarlton <leet@ami.com>
Change-Id: Ie7f21bbe71ffbb7650e17c8a063b121ffdf4e96e

show more ...

Changes in ipmi/test_ipmi_cold_reset.robot

Test Case

- Verify Cold Reset Impact On Sensor Threshold Via IPMI

Modified test case to avoid discrete sensor while selecting sensor

Changes in ipmi/test_ipmi_cold_reset.robot

Test Case

- Verify Cold Reset Impact On Sensor Threshold Via IPMI

Modified test case to avoid discrete sensor while selecting sensor random.
Added condition to avoid the sensor which didn't have single threshold value.
Added Ping Host keyword in Is BMC Operational Keyword on lib/ipmi_client.robot.

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

show more ...

Added new cases for IPMI Set User access payload.

Testcases Added:

1) Verify Set User Access Payload For Standard Payload SOL
Disable standard payload 1(SOL) using set user access payload co

Added new cases for IPMI Set User access payload.

Testcases Added:

1) Verify Set User Access Payload For Standard Payload SOL
Disable standard payload 1(SOL) using set user access payload command
and verify it in get user access payload command and also check
sol activate is not working.

2) Verify Set User Access Payload For Operator Privileged User
Verify set user access payload command is not accessbile for Operator
privileged User.

Moved below keywords from ipmi/test_ipmi_user.robot to lib/ipmi_client.robot
for reuse purpose.
Enable IPMI User And Verify
Create Random IPMI User
Delete Created User

Added IPMI raw cmd for Set user access payload.

Tested: Run robot ipmi/test_ipmi_paylaod.robot.

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

show more ...

Fix for loop exit directive

Report in community:
Verify IPMI User Deletion :: Delete user via IPMI and verify. | [31mFAIL[0m |
Parent suite setup failed:
'Break' is a reserved keyword.

Chan

Fix for loop exit directive

Report in community:
Verify IPMI User Deletion :: Delete user via IPMI and verify. | [31mFAIL[0m |
Parent suite setup failed:
'Break' is a reserved keyword.

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

show more ...

Fix issue #2187: multiple admins; root not UID 1

Along with the root user there may be other users with
ADMINISTRATOR privilege, and there is no guarantee that root will be
assigned user ID 1. But t

Fix issue #2187: multiple admins; root not UID 1

Along with the root user there may be other users with
ADMINISTRATOR privilege, and there is no guarantee that root will be
assigned user ID 1. But the test cases are hard-coded to expect root at
UID 1 and to have no other UIDs with ADMINISTRATOR privilege.
This can result in several test failures.

New keyword 'Determine Root User Id' dynamically determines the user ID
for root. New keyword 'Find Free User Id' finds a free user ID, avoiding
conflicts with all other enabled user IDs.

Signed-off-by: Gene Ratzlaff <generatz@gmail.com>
Change-Id: Iba7fe3afa11b6dd328e8536c2fbd136f3ee756f0

show more ...

Added new testcase for Get SDR info and Reserve SDR IPMI Commands

Modified Get SDR info testcase to verify supported commands dynamically,
rather than harcoded values.

Testcase Added:
- Test Re

Added new testcase for Get SDR info and Reserve SDR IPMI Commands

Modified Get SDR info testcase to verify supported commands dynamically,
rather than harcoded values.

Testcase Added:
- Test Reserve SDR repository.
- Verify Reserve SDR repository cmd functionality using get SDR cmd.
- Test Invalid reservation ID using Get SDR command.
- Verify Reserve SDR cmd after BMC reboot.

Tested Run robot ipmi/test_ipmi_sdr.robot

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

show more ...

Changes in ipmi/test_ipmi_sel_time.robot

Keyword renamed
- "Fetch Any Sensor From Sensor List" keyword was renamed to "Fetch One Threshold Sensor From Sensor List"

Signed-off-by: ganesanb <ganesan

Changes in ipmi/test_ipmi_sel_time.robot

Keyword renamed
- "Fetch Any Sensor From Sensor List" keyword was renamed to "Fetch One Threshold Sensor From Sensor List"

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

show more ...

Fix typo and misspell in code test case and documentation

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

Added new testcases for Cipher suite IPMI cmd.

Testcases covered in this script:
1) Verify cipher suite IPMI raw cmd for all available channels
including channel 14(current channel).

2) Ver

Added new testcases for Cipher suite IPMI cmd.

Testcases covered in this script:
1) Verify cipher suite IPMI raw cmd for all available channels
including channel 14(current channel).

2) Verify cipher suite IPMI raw cmd for invalid channels and expects
response code 0xcc.

3) Verify cipher suite IPMI raw cmd with one byte more and less for
valid channel and expects response code as 0xc7.

Added keyword on lib/ipmi_client.robot to check invalid ipmi cmd.

Added keyword to get invalid channel list in lib/bmc_network_utils.robot.

Tested: Run robot ipmi/test_ipmi_cipher.robot

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

show more ...

Added new cases for get channel authentication capabilities

Modified existing testcase for get channel authentication capabilities
via ipmi cmd to support multiple active channels.

Added testcase t

Added new cases for get channel authentication capabilities

Modified existing testcase for get channel authentication capabilities
via ipmi cmd to support multiple active channels.

Added testcase to verify channel auth cap ipmi cmd for invalid channels
Getting invalid channels list from /usr/share/ipmi-providers/channel_config.json
file.

Added testcase to verify ipmi raw cmd with one extra byte and
also raw cmd with one byte less

Tested: Run robot ipmi/test_ipmi_general.robot

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

show more ...

Changes in ipmi/test_ipmi_sel.robot

- Verify Clear SEL After Cold Reset

Removed ipmi raw cold reset command and added IPMI MC Reset Cold (RUN)

Modified Fetch Any Sensor From Sensor List keyword,

Changes in ipmi/test_ipmi_sel.robot

- Verify Clear SEL After Cold Reset

Removed ipmi raw cold reset command and added IPMI MC Reset Cold (RUN)

Modified Fetch Any Sensor From Sensor List keyword, Omitted discrete sensor and randomly selected
one threshold based sensor in lib/ipmi_client.robot.

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

show more ...

Fixed Typos for ipmi raw cmd and changed cold reset keywords

Changed from IPMI Std MC Reset Cold (off) to IPMI MC Reset Cold (run).

fixed typo according to ipmi_raw_cmd_table for below
param3_Pri

Fixed Typos for ipmi raw cmd and changed cold reset keywords

Changed from IPMI Std MC Reset Cold (off) to IPMI MC Reset Cold (run).

fixed typo according to ipmi_raw_cmd_table for below
param3_Primary_OS_Name to param3_Primary_Operating_System_Name
param4_OS_Name to param4_Operating_System_Name

Removed FFDC On Test Case Fail from Suite Teardown and
added in Test Teardown.

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

show more ...

Add new SEL test cases

Changes:
- Added validation for Reserver ID, Most Recent Timestamps,
Clear SEL and modifications in Add SEL Entry and Reserver ID

Testcases Modified:
- Ve

Add new SEL test cases

Changes:
- Added validation for Reserver ID, Most Recent Timestamps,
Clear SEL and modifications in Add SEL Entry and Reserver ID

Testcases Modified:
- Verify Reserve SEL
New Testcases:
- Verify Add SEL Entry For Any Random Sensor
- Verify IPMI SEL Most Recent Addition Timestamp
- Verify IPMI SEL Most Recent Erase Timestamp
- Verify Clear SEL With Invalid Reservation ID
- Verify Reservation ID Erasure Status
- Verify Clear SEL After Cold Reset

Verify Add SEL Entry For Any Random Sensor
- Modified with create SEL by sensor number and type taken i
from sensor list and sdr get. Verified the result with sel elist last 1

Verify Reserve SEL
- Added condition to check if the sel entry is empty.

Verify IPMI SEL Most Recent Addition Timestamp
- Gets Most Recent Addition Timestamp from IPMI Sel Info raw command
and validate against sel entry date. If no sel entry present, newi
sel entry will be created and validates the SEL date againt IPMI
Sel Most Recent Addition Timestamp.

Verify IPMI SEL Most Recent Erase Timestamp
- Clears the SEL and validates Most Recent Erase Timestamp from SEL
Info against bmc current time. Also verifies if Most Recent Addition
Timestamp is not available.

Verify Clear SEL With Invalid Reservation ID
- Creates two reservation id, stores first reservation id created and
tries to clear sel with stored reservation id. Once a new reservationi
id is created, previous reservation id generated will no longer be valid.

Verify Reservation ID Erasure Status
- Creates a reservation id, clears the sel using the reservation id created,
generate a new reservation id, check the erasure status using new reservation id

Verify Clear SEL After Cold Reset
- Creates a new reservation id, runs cold reset from lanplus, once bmc is booted,
checks whether the generated reservation id is accessible.

Tested: Run robot ipmi/test_ipmi_sel.robot

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

show more ...

Fixed few typos and changed external ipmi cmd to run via inband

Fixed typo error "SDR" to "sdr" in IPMI command.

Get Device Sdr info and Get Device SDR Ipmi command are executable
only from local

Fixed few typos and changed external ipmi cmd to run via inband

Fixed typo error "SDR" to "sdr" in IPMI command.

Get Device Sdr info and Get Device SDR Ipmi command are executable
only from local interface(system interface) so changed code to run
through Inband.

Added " IPMI MC Reset Cold (run)" keyword instead of ipmi raw command.

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

show more ...

Fix documentation and LineTooLong in the test suite

Fixes:
- Documetnation spacing for arguments
- Bellow LineTooLong changes

+ ipmi/test_ipmi_sel_time.robot
W: 508, 110: Line is too long (

Fix documentation and LineTooLong in the test suite

Fixes:
- Documetnation spacing for arguments
- Bellow LineTooLong changes

+ ipmi/test_ipmi_sel_time.robot
W: 508, 110: Line is too long (exceeds 110 characters) (LineTooLong)
W: 564, 110: Line is too long (exceeds 110 characters) (LineTooLong)
W: 582, 110: Line is too long (exceeds 110 characters) (LineTooLong)

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

show more ...

Fix return from keyword and correct variable name for index

Changes:
- Python lib/keyword Modify And Fetch Threshold return data of
list and dict in order. Made changes in test suite keyword

Fix return from keyword and correct variable name for index

Changes:
- Python lib/keyword Modify And Fetch Threshold return data of
list and dict in order. Made changes in test suite keyword
to accept return arguments correctly.
- Fix variable naming from list to dictionary

Error:
Resolving variable '${threshold_list['[2600, 'na', 'na', 12200]']}'
failed: SyntaxError: invalid syntax (<string>, line 1)

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

show more ...