diff options
author | Alpana Kumari <alpankum@in.ibm.com> | 2023-05-23 12:46:26 +0300 |
---|---|---|
committer | Adriana Kobylak <anoo@linux.ibm.com> | 2023-06-05 23:37:00 +0300 |
commit | 8528a12f154f508f1ede1a087b5be8b67ab3f5bb (patch) | |
tree | 4debc18a4539bb8f3db609b0d308ff93f9112fb0 /meta-ibm/recipes-phosphor/inventory/phosphor-inventory-manager | |
parent | 595b4421de8ea135984cfe5703f7abc227929b7c (diff) | |
download | openbmc-8528a12f154f508f1ede1a087b5be8b67ab3f5bb.tar.xz |
meta-ibm: Add LED endpoints for System
LED endpoints with Identifying, Identified_by for System was missing in
Association json,as a result of that LocationIndicatorActive for system
was null in redfish GET output.
Also, from GUI, System identify LED was not functioning.
If we try to set it, it goes away once we refresh the page,and no impact
on led group.
So this has been addressed for P10bmc Rainier and Everest system.
Tested:-
-Toggling reflects on LocationIndicatorActive as well as LED state of
associated LED group.
-It Works fine with redfish curl command as well as GUI button.
$ curl -k -H "X-Auth-Token: $bmc_token" -X PATCH https://${bmc}/redfish/v1/Systems/system -d '{"LocationIndicatorActive": true}'
$
$ curl -k -H "X-Auth-Token: $bmc_token" -X GET https://${bmc}/redfish/v1/Systems/system |grep "LocationIndicatorActive"
% Total % Received % Xferd Average Speed Time Time Time Current
Dload Upload Total Spent Left Speed
100 6262 100 6262 0 0 42026 0 --:--:-- --:--: "LocationIndicatorActive": true,
- It shows up the result on hardware as below-
~# busctl introspect xyz.openbmc_project.LED.GroupManager /xyz/openbmc_project/led/groups/enclosure_identify
NAME TYPE SIGNATURE RESULT/VALUE FLAGS
org.freedesktop.DBus.Introspectable interface - - -
.Introspect method - s -
org.freedesktop.DBus.Peer interface - - -
.GetMachineId method - s -
.Ping method - - -
org.freedesktop.DBus.Properties interface - - -
.Get method ss v -
.GetAll method s a{sv} -
.Set method ssv - -
.PropertiesChanged signal sa{sv}as - -
xyz.openbmc_project.Led.Group interface - - -
.Asserted property b true emits-change writable
~# cat /var/lib/phosphor-led-manager/savedGroups
{
"value0": [
"/xyz/openbmc_project/led/groups/bmc_booted",
"/xyz/openbmc_project/led/groups/enclosure_identify"
]
}#
Change-Id: I20dd57cc0c4edb0e28039ca9a276075914adc9f6
Signed-off-by: Alpana Kumari <alpankum@in.ibm.com>
Diffstat (limited to 'meta-ibm/recipes-phosphor/inventory/phosphor-inventory-manager')
3 files changed, 33 insertions, 0 deletions
diff --git a/meta-ibm/recipes-phosphor/inventory/phosphor-inventory-manager/p10bmc/ibm,everest_associations.json b/meta-ibm/recipes-phosphor/inventory/phosphor-inventory-manager/p10bmc/ibm,everest_associations.json index 95612b9a66..dd197c1b8a 100644 --- a/meta-ibm/recipes-phosphor/inventory/phosphor-inventory-manager/p10bmc/ibm,everest_associations.json +++ b/meta-ibm/recipes-phosphor/inventory/phosphor-inventory-manager/p10bmc/ibm,everest_associations.json @@ -383,6 +383,17 @@ "/xyz/openbmc_project/led/groups/partition_system_attention_indicator", "/xyz/openbmc_project/led/groups/platform_system_attention_indicator" ] + }, + { + "types": + { + "rType": "identified_by", + "fType": "identifying" + }, + "paths": + [ + "/xyz/openbmc_project/led/groups/enclosure_identify" + ] } ] }, diff --git a/meta-ibm/recipes-phosphor/inventory/phosphor-inventory-manager/p10bmc/ibm,rainier-2u_associations.json b/meta-ibm/recipes-phosphor/inventory/phosphor-inventory-manager/p10bmc/ibm,rainier-2u_associations.json index 1eabb3afa7..2b85f9deff 100644 --- a/meta-ibm/recipes-phosphor/inventory/phosphor-inventory-manager/p10bmc/ibm,rainier-2u_associations.json +++ b/meta-ibm/recipes-phosphor/inventory/phosphor-inventory-manager/p10bmc/ibm,rainier-2u_associations.json @@ -430,6 +430,17 @@ "/xyz/openbmc_project/led/groups/partition_system_attention_indicator", "/xyz/openbmc_project/led/groups/platform_system_attention_indicator" ] + }, + { + "types": + { + "rType": "identified_by", + "fType": "identifying" + }, + "paths": + [ + "/xyz/openbmc_project/led/groups/enclosure_identify" + ] } ] }, diff --git a/meta-ibm/recipes-phosphor/inventory/phosphor-inventory-manager/p10bmc/ibm,rainier-4u_associations.json b/meta-ibm/recipes-phosphor/inventory/phosphor-inventory-manager/p10bmc/ibm,rainier-4u_associations.json index 3debdafeb0..a43ee5fa62 100644 --- a/meta-ibm/recipes-phosphor/inventory/phosphor-inventory-manager/p10bmc/ibm,rainier-4u_associations.json +++ b/meta-ibm/recipes-phosphor/inventory/phosphor-inventory-manager/p10bmc/ibm,rainier-4u_associations.json @@ -436,6 +436,17 @@ "/xyz/openbmc_project/led/groups/partition_system_attention_indicator", "/xyz/openbmc_project/led/groups/platform_system_attention_indicator" ] + }, + { + "types": + { + "rType": "identified_by", + "fType": "identifying" + }, + "paths": + [ + "/xyz/openbmc_project/led/groups/enclosure_identify" + ] } ] }, |