commit c32434a3acd8f72de9cbcdd20c90669e8d507609
author Yi Hu <yih@supermicro.com> Thu Jan 11 17:33:10 2024 -0800
committer George Keishing <gkeishin@gmail.com> Mon Apr 01 04:10:42 2024 +0000
tree 97429f691f4e27967652833c5e3eea7e01e120eb
parent 8a86c69d7d14a6f064919a5f96aecec756370a9c

Set system, chassis, and bmc as global variables

Changes:
  - Add SYSTEM_ID as a global variable and set to default value system
  - Replace redfish/v1/Systems/system to redfish/v1/Systems/SYSTEM_ID
  - Replace redfish/v1/Chassis/chassis to redfish/v1/Chassis/CHASSIS_ID
  - Replace redfish/v1/Managers/bmc to redfish/v1/Managers/MANAGER_ID

Tested:
  - Tested on sandbox with the changes with sample test run with
    sample test ID runs
    1. without SYSTEM_ID provided
    2. with -v SYSTEM_ID:system1
    3. with export SYSTEM_ID=""
    3. with export SYSTEM_ID="system2"

Change-Id: Ifc412b20038cef1eba632caaa6c300be328259bd
Signed-off-by: Wilson Suen <wilsons@supermicro.com>
Signed-off-by: George Keishing <gkeishin@in.ibm.com>

openpower/mpipl/test_mpipl_basic.robot

diff --git a/openpower/mpipl/test_mpipl_basic.robot b/openpower/mpipl/test_mpipl_basic.robot
index ca1498b..5fd0507 100644
--- a/openpower/mpipl/test_mpipl_basic.robot
+++ b/openpower/mpipl/test_mpipl_basic.robot
@@ -143,7 +143,7 @@
     #       "Name": "System Dump Entry",
     #       "OEMDiagnosticDataType": "System"
     #   }
-    ${sys_dump}=  Redfish.Get Properties  /redfish/v1/Systems/system/LogServices/Dump/Entries
+    ${sys_dump}=  Redfish.Get Properties  /redfish/v1/Systems/${SYSTEM_ID}/LogServices/Dump/Entries
     Log To Console  System dump generated: ${sys_dump['Members@odata.count']}
     Should Be True  ${sys_dump['Members@odata.count']} == 1  msg=No system dump generated.
 

openpower/pel/test_bmc_pel.robot

diff --git a/openpower/pel/test_bmc_pel.robot b/openpower/pel/test_bmc_pel.robot
index ec51e45..5ba76a6 100644
--- a/openpower/pel/test_bmc_pel.robot
+++ b/openpower/pel/test_bmc_pel.robot
@@ -165,9 +165,9 @@
     #    [Serial Number]:                              ABCDEFG    <---- Serial number
     #    [Sub-section type]:                           0
 
-    ${redfish_machine_model}=  Redfish.Get Attribute  /redfish/v1/Systems/system/  Model
+    ${redfish_machine_model}=  Redfish.Get Attribute  /redfish/v1/Systems/${SYSTEM_ID}/  Model
     ${redfish_machine_model}=  Replace String Using Regexp  ${redfish_machine_model}  ^0+  ${EMPTY}
-    ${redfish_serial_number}=  Redfish.Get Attribute  /redfish/v1/Systems/system/  SerialNumber
+    ${redfish_serial_number}=  Redfish.Get Attribute  /redfish/v1/Systems/${SYSTEM_ID}/  SerialNumber
     ${redfish_serial_number}=  Replace String Using Regexp  ${redfish_serial_number}  ^0+  ${EMPTY}
 
     Valid Value  pel_machine_type_model  ['${redfish_machine_model}']
@@ -267,7 +267,7 @@
     #    [BMC Event Log Id]:           341      <---- BMC event log id value
     #    [Committed at]:               03/25/1920 12:06:22
 
-    ${redfish_event_logs}=  Redfish.Get Properties  /redfish/v1/Systems/system/LogServices/EventLog/Entries
+    ${redfish_event_logs}=  Redfish.Get Properties  /redfish/v1/Systems/${SYSTEM_ID}/LogServices/EventLog/Entries
 
     # Example of redfish_event_logs output:
     # redfish_event_logs:

openpower/test_timed_power_on.robot

diff --git a/openpower/test_timed_power_on.robot b/openpower/test_timed_power_on.robot
index 0fe7517..2f24511 100644
--- a/openpower/test_timed_power_on.robot
+++ b/openpower/test_timed_power_on.robot
@@ -119,7 +119,7 @@
     # Description of argument(s):
     # power_policy_mode           BIOS attribute value. E.g. "Stay On", "Automatic".
 
-    Redfish.Patch  /redfish/v1/Systems/system/Bios/Settings
+    Redfish.Patch  /redfish/v1/Systems/${SYSTEM_ID}/Bios/Settings
     ...  body={"Attributes":{"pvm_system_power_off_policy": "${power_policy_mode}"}}
     ...  valid_status_codes=[${HTTP_OK}]