Thermal Metrics / Temperatures

[gtmills]

Part of the new Power / Thermal

[gtmills]

implement this schema: https://redfish.dmtf.org/schemas/v1/ThermalMetrics.v1_0_0.json

[ChicagoDuan]

Hi @gtmills,
There's some questions. Please see the example response about TemperatureSummaryCelsiussuch in ThermalMetrics schema . how do we know the sensor name of Internal, It is different in different machines, may be it is "CPU1_Temp" or "cpu1Temp". So how do we hardcode the DataSourceUri ? Thanks.

"@odata.type": "#ThermalMetrics.v1_0_0.ThermalMetrics",
"Id": "ThermalMetrics",
"Name": "Chassis Thermal Metrics",
"TemperatureSummaryCelsius": {
    "Internal": {
        "Reading": 39,
        "DataSourceUri": "/redfish/v1/Chassis/1U/Sensors/CPU1Temp"
},
    "Intake": {
        "Reading": 24.8,
        "DataSourceUri": "/redfish/v1/Chassis/1U/Sensors/IntakeTemp"
},
    "Ambient": {
        "Reading": 22.5,
        "DataSourceUri": "/redfish/v1/Chassis/1U/Sensors/AmbientTemp"
},
    "Exhaust": {
        "Reading": 40.5,
        "DataSourceUri": "/redfish/v1/Chassis/1U/Sensors/ExhaustTemp"
}
},

[gtmills]

@ChicagoDuan I think we need to discuss this with the community; we need some way to say "this is an Internal temperatureSummary sensor" or "this is an Exhaust temperatureSummarysensor" ?
Maybe associations?

[zhanghch05]

review:
https://gerrit.openbmc-project.xyz/c/openbmc/bmcweb/+/41121

[gtmills]

Can we pull this one in downstream?

[lxwinspur]

Can we pull this one in downstream?

Sure, will do

[lxwinspur]

Can we pull this one in downstream?

I checked this patch and it was pulled downstream four weeks ago.
ibm-openbmc/bmcweb@aaba73c

[gtmills]

Cool. Thanks!
We use "Code Review in Driver" to mean it is downstream.

[gtmills]

Still need https://gerrit.openbmc.org/c/openbmc/bmcweb/+/41121

[lxwinspur]

Still need https://gerrit.openbmc.org/c/openbmc/bmcweb/+/41121

Updated