| commit | b001a52b39078057be61f9e812ba35c6cbc574cf | [log] [tgz] |
|---|---|---|
| author | Myung Bae <myungbae@us.ibm.com> | Thu Jun 27 08:18:31 2024 -0700 |
| committer | Adriana Kobylak <anoo@us.ibm.com> | Fri Jun 28 18:41:42 2024 +0000 |
| tree | 549a0d2ca24f7dd705251c11557c4d795a239375 | |
| parent | 77c94fa9ba38f7e1f827766e734587a239520cd6 [diff] |
meta-ibm: p10bmc: disable deprecated power-thermal
Redfish service validator reports
```
1 err.Edm.Decimal errors in /redfish/v1/Chassis/chassis/Power#/PowerSupplies/0
1 failProp errors in /redfish/v1/Chassis/chassis/Power#/PowerSupplies/0
1 fails errors in /redfish/v1/Chassis/chassis/Power#/PowerSupplies/0
```
```
curl -k -X GET https://${bmc}/redfish/v1/Chassis/chassis/Power
{
...
"PowerSupplies": [
{
"@odata.id": "/redfish/v1/Chassis/chassis/Power#/PowerSupplies/0",
...
"PowerInputWatts": -5.5, <---
...
},
```
This power-thermal support had been being deprecated and thus this
commit is to disable the deprecated power-thermal.
With this, some information (PowerCap) would be missing on webui-vue[1].
Webui-vue would need to switch over to use these[2].
Tested:
- The deprecated Power is no longer shown from
`curl -k -X GET https://${bmc}/redfish/v1/Chassis/chassis`
- Redfish Service Validator passes on Power
[1] https://github.com/openbmc/webui-vue/blob/8841b7d463a5272a87faaa14cb103f778a772770/src/store/modules/ResourceManagement/PowerControlStore.js#L41
[2] https://gerrit.openbmc.org/q/topic:%22redfish-EnvironmentMetrics%22
Change-Id: I86e46ef74cdce1bc7dea994dced93bb432082774
Signed-off-by: Myung Bae <myungbae@us.ibm.com>
diff --git a/meta-ibm/recipes-phosphor/interfaces/bmcweb_%.bbappend b/meta-ibm/recipes-phosphor/interfaces/bmcweb_%.bbappend index 0116afa..7a3455d 100644 --- a/meta-ibm/recipes-phosphor/interfaces/bmcweb_%.bbappend +++ b/meta-ibm/recipes-phosphor/interfaces/bmcweb_%.bbappend
@@ -11,6 +11,7 @@ EXTRA_OEMESON:append:p10bmc = " \ -Dkvm=disabled \ -Dvm-websocket=disabled \ + -Dredfish-allow-deprecated-power-thermal=disabled \ " PACKAGECONFIG:remove:p10bmc = "mutual-tls-auth"