Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support new ThermalSubsystem, PowerSubsystem, Fan, PowerSupply for OCP Server Profile #74

Open
AmyChang-yh opened this issue Apr 10, 2024 · 3 comments

Comments

@AmyChang-yh
Copy link

Same reason as #25.

Redfish in 2020.4 added PowerSubsystem, ThermalSubsystem, Fan, PowerSupply, PowerSupplyMetrics
https://www.dmtf.org/sites/default/files/Redfish_Release_2020.4_Overview.pdf

#25 can fix the issue on Baseline Profile, could we apply the change to Server Profile as well?

@jcleung5549
Copy link
Contributor

jcleung5549 commented Apr 10, 2024

That can be done upon request of the Server Project, since the Server Profile references the Baseline Profile. The Server Project specifies to which profile an implement needs to conform.

See the Profile Release Process

@AmyChang-yh
Copy link
Author

I tried to modify the parameter of OCPBaselineHardwareManagement MinVersion to 1.1.0 in OCPServerHardwareManagement.v1_1_0.json, but the Power and Thermal errors still exist by running Redfish-Interop-Validator.

ERROR - 1 fail.Comparison.Thermal.Temperatures.PhysicalContext errors in Service Level Requirements
ERROR - 1 fail.Power.ReadRequirement errors in Service Level Requirements
ERROR - 1 fail.Thermal.ReadRequirement errors in Service Level Requirements

I'm sure the Redfish-Interop-Validator had included the correct profiles because I could see the messages below.

Profile Hashes:
INFO - profile: OCPServerHardwareManagement 1.1.0, dict md5 hash: ee7fd33a778ce096020059f0dad4a15e
INFO - profile: OCPBaselineHardwareManagement 1.1.0, dict md5 hash: 94ebe02b495cecb5798a5df97b269ea6

It seems a few changes are needed for Server Profile as well.

@jcleung5549
Copy link
Contributor

Thank you for testing this draft profile. We will look into it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants