commit e5ae9c1c8787f9d8edf6f46acc7cdb83004f5125
author George Liu <liuxiwei@inspur.com> Tue Nov 16 10:31:23 2021 +0800
committer Ed Tanous <ed@tanous.net> Wed Mar 27 18:24:57 2024 +0000
tree f7ce794d1084ada254c57271f809aa7f6952cc6d
parent d0eb0e7303744d03aaa6f994bbbcdd6dc92bab13

Update indicator LED verification

Extend the hasIndicatorLed array and add
xyz.openbmc_project.Inventory.Item.Chassis interface.

Tested:
```
curl -k https://$bmc/redfish/v1/Chassis/chassis
{
  "@odata.id": "/redfish/v1/Chassis/chassis",
  "@odata.type": "#Chassis.v1_22_0.Chassis",
  "Actions": {
    "#Chassis.Reset": {
      "@Redfish.ActionInfo": "/redfish/v1/Chassis/chassis/ResetActionInfo",
      "target": "/redfish/v1/Chassis/chassis/Actions/Chassis.Reset"
    }
  },
  "ChassisType": "RackMount",
  "Id": "chassis",
  "Links": {
    "ComputerSystems": [
      {
        "@odata.id": "/redfish/v1/Systems/system"
      }
    ],
    "ManagedBy": [
      {
        "@odata.id": "/redfish/v1/Managers/bmc"
      }
    ]
  },
  "Location": {
    "PartLocation": {
      "ServiceLabel": "U78DA.ND0.WZS004K"
    }
  },
  "IndicatorLED": "Off",
  "LocationIndicatorActive": false,
  "Manufacturer": "",
  "Model": "23",
  "Name": "chassis",
  "PCIeDevices": {
    "@odata.id": "/redfish/v1/Systems/system/PCIeDevices"
  },
  "PartNumber": "",
  "Power": {
    "@odata.id": "/redfish/v1/Chassis/chassis/Power"
  },
  "PowerState": "Off",
  "Sensors": {
    "@odata.id": "/redfish/v1/Chassis/chassis/Sensors"
  },
  "SerialNumber": "",
  "Status": {
    "Health": "OK",
    "HealthRollup": "OK",
    "State": "StandbyOffline"
  },
  "Thermal": {
    "@odata.id": "/redfish/v1/Chassis/chassis/Thermal"
  }
}
```

Signed-off-by: George Liu <liuxiwei@ieisystem.com>
Change-Id: I02e77d56e555f9aee3f76015baeebbf1f4a292ab

redfish-core/lib/chassis.hpp

diff --git a/redfish-core/lib/chassis.hpp b/redfish-core/lib/chassis.hpp
index 6d5e587..02bd7a4 100644
--- a/redfish-core/lib/chassis.hpp
+++ b/redfish-core/lib/chassis.hpp
@@ -526,7 +526,8 @@
         const std::string& connectionName = connectionNames[0].first;
 
         const std::vector<std::string>& interfaces2 = connectionNames[0].second;
-        const std::array<const char*, 2> hasIndicatorLed = {
+        const std::array<const char*, 3> hasIndicatorLed = {
+            "xyz.openbmc_project.Inventory.Item.Chassis",
             "xyz.openbmc_project.Inventory.Item.Panel",
             "xyz.openbmc_project.Inventory.Item.Board.Motherboard"};
 
@@ -735,7 +736,8 @@
             const std::vector<std::string>& interfaces3 =
                 connectionNames[0].second;
 
-            const std::array<const char*, 2> hasIndicatorLed = {
+            const std::array<const char*, 3> hasIndicatorLed = {
+                "xyz.openbmc_project.Inventory.Item.Chassis",
                 "xyz.openbmc_project.Inventory.Item.Panel",
                 "xyz.openbmc_project.Inventory.Item.Board.Motherboard"};
             bool indicatorChassis = false;