| commit | 5c6221ac85ce72a612f7001246fb5a7ab521c7a5 | [log] [tgz] |
|---|---|---|
| author | Gunnar Mills <gmills@us.ibm.com> | Fri Feb 22 11:24:29 2019 -0600 |
| committer | Ed Tanous <ed.tanous@intel.com> | Mon Mar 11 22:12:41 2019 +0000 |
| tree | 574a6c700a34c4b7e678890bf370e35739eaaaf0 | |
| parent | 6f4fd47925f33d45b31bcf61e5eac2ee22913f77 [diff] |
Fix IndicatorLED "Blinking" value Blink is not a valid value for Redfish IndicatorLED. Blinking is though. Changed. See: https://redfish.dmtf.org/schemas/ComputerSystem.v1_6_0.json http://redfish.dmtf.org/schemas/v1/Resource.json#/definitions/IndicatorLED LED D-Bus interface has the value "Blink". Updated. https://github.com/openbmc/phosphor-dbus-interfaces/blob/master/xyz/openbmc_project/Led/Physical.interface.yaml#L40 Tested: Limited. Change-Id: I183b72f06f2dddf6a37592893e841d810d06a0d6 Signed-off-by: Gunnar Mills <gmills@us.ibm.com>
diff --git a/redfish-core/lib/systems.hpp b/redfish-core/lib/systems.hpp index 5f10f16..737aa3a 100644 --- a/redfish-core/lib/systems.hpp +++ b/redfish-core/lib/systems.hpp
@@ -1211,10 +1211,9 @@ { dbusLedState = "xyz.openbmc_project.Led.Physical.Action.Lit"; } - else if (*indicatorLed == "Blink") + else if (*indicatorLed == "Blinking") { - dbusLedState = - "xyz.openbmc_project.Led.Physical.Action.Blinking"; + dbusLedState = "xyz.openbmc_project.Led.Physical.Action.Blink"; } else if (*indicatorLed == "Off") {