Enable redfish-new-powersubsystem-thermalsubsystem
This is just the default, but enable
redfish-new-powersubsystem-thermalsubsystem. As discussed on discord 4
companies enabled it upstream and I see a few more downstream forks
enabling it as well.
This does enable all sensors in the sensor collection, this is following
Redfish as described in the new thermalSubsystem and powerSubsystem doc
the sensor collection should contain all sensors that are associated
with that chassis. [1]
[1] https://redfishforum.com/thread/190/sensorcollection-contain-all-sensors-chassis
Redfish release 2020.4 was the new powersubsystem, thermalsubsystem
schemas. Redfish has continued to add to them, deprecate the old, and
stated this is the new schemas are the future direction.
Leave the old redfish-allow-deprecated-power-thermal alone (enabled),
this allows for an easier client transition. Also, it would be best if
the few outstanding new powersubsystem, thermalsubsystem children
resources could get in first. Added a warning about disabling
redfish-allow-deprecated-power-thermal in June 2024.
Tested: See these APIs, a validator run with both of these enabled
(average (2 runs) 6:06) and a validator run with just
redfish-new-powersubsystem-thermalsubsystm (average 2 runs 5:52).
3.8% increase. I think this increase is worth it for the client
compatibility.
Change-Id: Ideb0f1999289b11b80ee0b8288e7ce53de0a7433
Signed-off-by: Gunnar Mills <gmills@us.ibm.com>
diff --git a/meson_options.txt b/meson_options.txt
index 584ca70..8441770 100644
--- a/meson_options.txt
+++ b/meson_options.txt
@@ -226,11 +226,10 @@
option(
'redfish-new-powersubsystem-thermalsubsystem',
type: 'feature',
- value: 'disabled',
+ value: 'enabled',
description: '''Enable/disable the new PowerSubsystem, ThermalSubsystem,
and all children schemas. This includes displaying all
- sensors in the SensorCollection. At a later date, this
- feature will be defaulted to enabled.'''
+ sensors in the SensorCollection.'''
)
option(
@@ -238,7 +237,8 @@
type: 'feature',
value: 'enabled',
description: '''Enable/disable the old Power / Thermal. The default
- condition is allowing the old Power / Thermal.'''
+ condition is allowing the old Power / Thermal. This
+ will be disabled by default June 2024. '''
)
option(