IPMI Watchdog - Gerrit Reviewer Comment Changes

Create a new branch for latest IPMI Watchdog changes.

Add changes from gerrit reviewer comments.

Rebase from gerrit/master and commit

Abandoned pre

IPMI Watchdog - Gerrit Reviewer Comment Changes

Create a new branch for latest IPMI Watchdog changes.

Add changes from gerrit reviewer comments.

Rebase from gerrit/master and commit

Abandoned previous IPMI Watchdog MR and branches due to MR being
behind master branch by many commits and having a merge conflict.

Change-Id: If9bacf307982870c5d0e2dbe56a14145dc0ee4ce
Signed-off-by: Lee Tarlton <leet@ami.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 ...

Update DMTF Refish use case checker test suite

Changes:
- Add logic to print errors in test suite
- Update the directory where log is collected
- Fix typo for IPMI suite

Resolves o

Update DMTF Refish use case checker test suite

Changes:
- Add logic to print errors in test suite
- Update the directory where log is collected
- Fix typo for IPMI suite

Resolves openbmc/openbmc-test-automation#2177

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

show more ...

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

Fix IPMI verify VLAN on different interface

The test case "Create VLAN When LAN And VLAN Exist With IP Address
Configured" create VLAN on default interface eth0 and try to verify VLAN
IP setting on

Fix IPMI verify VLAN on different interface

The test case "Create VLAN When LAN And VLAN Exist With IP Address
Configured" create VLAN on default interface eth0 and try to verify VLAN
IP setting on "activate" interface. Pass the interface parameter to make
sure the VLAN created and verified on same interface.

Signed-off-by: Brian Ma <chma0@nuvoton.com>
Change-Id: Ic982db8dfdcb0b73533e3d3e9f7df239be6d36f4

show more ...

DMTF redfish system path logic update in test suites

Changes:
- Dynamincally find the logical id and use

Resolves openbmc/openbmc-test-automation#2172

Change-Id: I9af1cc6a3ad4b7ad29270b9b80

DMTF redfish system path logic update in test suites

Changes:
- Dynamincally find the logical id and use

Resolves openbmc/openbmc-test-automation#2172

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

show more ...

Fix common code in line with standard coding expectation

Changes fix for the following:

Line has trailing whitespace (TrailingWhitespace)
Too many trailing blank lines (TrailingBlankLines)
Line is

Fix common code in line with standard coding expectation

Changes fix for the following:

Line has trailing whitespace (TrailingWhitespace)
Too many trailing blank lines (TrailingBlankLines)
Line is too long (exceeds 110 characters) (LineTooLong)

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

show more ...

Add Power Cycle for IPMI Get BIOS POST Code command

Add power cycle step for test Test Get BIOS POST Code
via IPMI Raw Command After Power Cycle.
This command was accidently left out.
Change for lon

Add Power Cycle for IPMI Get BIOS POST Code command

Add power cycle step for test Test Get BIOS POST Code
via IPMI Raw Command After Power Cycle.
This command was accidently left out.
Change for longer host_reboot_time, and power host
back on at end of test suite.

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

show more ...

Added test to verify modified IPMI username

Test cases added:

- Modify_IPMI_User

It will create a new user and set access, change previlege

and change the username and verify the modified use

Added test to verify modified IPMI username

Test cases added:

- Modify_IPMI_User

It will create a new user and set access, change previlege

and change the username and verify the modified username is accessible.

verifies modified IPMI user is communicating via IPMI.

Tested: Run robot "Modify_IPMI_User" of ipmi/test_ipmi_user.robot

Signed-off-by: Chithra Gopinathan <chithrag@ami.com>
Change-Id: I712da488422084c857fb54fb70fdac7804add656

show more ...

Rebased for IPMI Get BIOS POST Code

New script for IPMI Get BIOS POST Code command

Add new script for retrieving BIOS POST codes via IPMI interface.
The script tests the command while Host is power

Rebased for IPMI Get BIOS POST Code

New script for IPMI Get BIOS POST Code command

Add new script for retrieving BIOS POST codes via IPMI interface.
The script tests the command while Host is powered on, after a
Host power cycle, and after a Host power off.

Note: The power off test case fails since in the test image
for this feature, chassis power off is not implemented. Also, lanplus
is not working with the test image.

Test results uploaded in the MR.

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

show more ...

Fix test case tag names from multiple suites

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

Fix IPMI lan interface count loop

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

IPMI SOL log file path default

Changes:
- Default the SOL in user execution directory logs/
- Update Power on and off to use IPMI methods

This changes is to avoid leaving files under /tmp/ wh

IPMI SOL log file path default

Changes:
- Default the SOL in user execution directory logs/
- Update Power on and off to use IPMI methods

This changes is to avoid leaving files under /tmp/ which will
eventually flush on the reboot but user using server running
the test hardly reboot the system. The existing scheme works
ok if running on local or docker instances but still not a
good idea to dump onto /tmp/.

Resolves openbmc/openbmc-test-automation#2132

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

show more ...

Add method to create error log using busctl

Resolves openbmc/openbmc-test-automation#2133

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

Fix test tag typo in IPMI

Resolves openbmc/openbmc-test-automation#2129

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

Modified hard-code when get FRU info

Modify hard-code to global variable when get FRU info.

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

Added IPMI chassis identify testcases with verification via Redfish

- Verify Chassis Identify via IPMI
- Verify Chassis Identify Off And Force Identify On via IPMI

Change-Id: Ieea260199e9754cccbbd0

Added IPMI chassis identify testcases with verification via Redfish

- Verify Chassis Identify via IPMI
- Verify Chassis Identify Off And Force Identify On via IPMI

Change-Id: Ieea260199e9754cccbbd0e3b781d684d7f146c19
Signed-off-by: Ashwini Chandrappa <Ashwini.Chandrappa777@ibm.com>

show more ...

Added test to continuous check IPMI command execution

Change-Id: I71cc5b15aafb37cf9acc479d96292eb0f3a7d6f8
Signed-off-by: Ashwini Chandrappa <Ashwini.Chandrappa777@ibm.com>

ipmi: sdr: add one more condition for sdr dimm is not present case

Symptom:
Verify SDR keyword got failed then cause below test cases FAIL.
Test DIMM SDR Info At Power Off | FAIL |
Test DIMM SDR Inf

ipmi: sdr: add one more condition for sdr dimm is not present case

Symptom:
Verify SDR keyword got failed then cause below test cases FAIL.
Test DIMM SDR Info At Power Off | FAIL |
Test DIMM SDR Info At Power On | FAIL |

Root cause:
According ipmitool sdr elist for dimm result as below:
When dimm0 with real ram on it the result is "Presence Detected" string,
dimm1 and dimm2 without any ram on it the result is empty string.

For example on Olympus: ipmitool sdr elist
dimm0 | 72h | ok | 32.25 | Presence Detected
dimm1 | 73h | ok | 32.26 |
dimm2 | 74h | ok | 32.27 |

However, auto test case keyword "Verify SDR" didn't consider
result is empty string, just only consider result is "Disabled"
string. Thus, cause test case print "Invalid Presence" then test got failed.

Solution:
Add one more condition about result is empty string for sdr dimm is not present

Tested:
Run robot test -v Test_DIMM_SDR_Info_At_Power_Off/On ipmi/test_ipmi_sdr.robot
Test DIMM SDR Info At Power Off | PASS |
Test DIMM SDR Info At Power On | PASS |

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

show more ...

Add Redfish login session for the suites

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

Add boot utils resource for using Redfish power on/off

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

Modify hard-code when set a network config

Modify hard-code to global variable when set a network configuration.

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

Modify hard-code when set a network config

Modify hard-code to global variable when set a network configuration.

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

show more ...

ipmi sensor: fixd error reading ReadingCelsius

Signed-off-by: Joseph Liu <kwliu@nuvoton.com>
Signed-off-by: Tim Lee <timlee660101@gmail.com>
Change-Id: I4218f18a3d28310f4a5161e8a67cf8b98e40f98e

ipmi: add test for FRU device name

Added basic test for FRU device name. It executes basic FRU command
and search in result for given device name (passed as command line
parameter FRU_NAME:'Device N

ipmi: add test for FRU device name

Added basic test for FRU device name. It executes basic FRU command
and search in result for given device name (passed as command line
parameter FRU_NAME:'Device Name'). Test pass if device found.

Test: Verified this on target.
robot -v OPENBMC_HOST:73.11.197.245 -v FRU_NAME:S7106GM2NR
ipmi/test_ipmi_fru_device.robot

Signed-off-by: Vijay Khemka <vijaykhemkalinux@gmail.com>
Change-Id: I6bc8b46bf3f5addf5f713a5b4402b3557321fafe

show more ...

SoL tests: skip power operations when they aren't needed

Skip 'Redfish Hard Power Off' command if a system is
already in the powered off state.

Signed-off-by: Konstantin Aladyshev <aladyshev22@gmai

SoL tests: skip power operations when they aren't needed

Skip 'Redfish Hard Power Off' command if a system is
already in the powered off state.

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

show more ...