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

PLDM broken due to https://gerrit.openbmc.org/c/openbmc/bios-settings-mgr/+/63061 #4

Open
sagisin opened this issue Nov 20, 2023 · 0 comments

Comments

@sagisin
Copy link

sagisin commented Nov 20, 2023

https://gerrit.openbmc.org/c/openbmc/bios-settings-mgr/+/63061 - this was a commit for if the BIOS attribute is of type ENUM then each value should be mapped to a value name. if this was done only for ENUM attributes, why was the generic baseBIOSTable structure changed for all types of attributes - string, integer and enum attrs?
Also, could you help me understand how does the redfish output looks for string and integer attributes?

As part of this change in the bios-settings-mgr the baseBIOSTable type has changed from a{s(sbsssvva(sv))} to a{s(sbsssvva(svs))}, so the pldm master is broken due to this, as PLDM updates the baseBIOSTABLE of the bios-settings-mgr

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

No branches or pull requests

1 participant