Redfish : Look for OperationalStatus for the Health and HealthRollUp in health.hpp

[ChicagoDuan]

health.hpp should be modified to look for OperationalStatus for the Health and HealthRollUp.
Ideally, all resources would then use health.hpp.
It might make sense to rename health.hpp to status.hpp and have it do the health, healthrollup, and state.
#2795 (comment)

Step 1 :
We should push a paragraph to : https://github.com/openbmc/docs/blob/919a7b6816a5f16aa72d298e81e0756d95d5031e/designs/redfish-health-rolllup.md#proposed-design
E.g.

# Individual Resource Health 
All resources should look at OperationalStatus  to determine the Health 

Step 2:
Modify the health.hpp to look for OperationalStatus for the Health and HealthRollUp.

[vishwabmc]

@ChicagoDuan Isn't health.cpp already looking at OperationalStatus to populate Health ?.

or , may be processor / DIMMs / Fans, individually have hooks on OperationalStatus and then use Health also ?. Because, I can today set health of Fan using OperationalStatus

https://github.com/openbmc/bmcweb/blob/master/redfish-core/lib/processor.hpp#L76

https://github.com/openbmc/bmcweb/blob/master/redfish-core/lib/sensors.hpp#L801

@gtmills @lkammath

[ChicagoDuan]

@ChicagoDuan Isn't health.cpp already looking at OperationalStatus to populate Health ?.

or , may be processor / DIMMs / Fans, individually have hooks on OperationalStatus and then use Health also ?. Because, I can today set health of Fan using OperationalStatus

https://github.com/openbmc/bmcweb/blob/master/redfish-core/lib/processor.hpp#L76

https://github.com/openbmc/bmcweb/blob/master/redfish-core/lib/sensors.hpp#L801

@gtmills @lkammath

Now health.hpp doesn't look for OperationalStatus.
You can today set health of Fan using OperationalStatus, because it not uses health.hpp

[ChicagoDuan]

Design document: https://gerrit.openbmc-project.xyz/c/openbmc/docs/+/43485

[gtmills]

This isn't complete, can we move all bmcweb resources to use OperationalStatus?