[SONiC BMC] Get the BMC eeprom from Redis - #747
Open
benle7 wants to merge 1 commit into
Open
Conversation
Collaborator
|
/azp run |
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
#### Description Extend `BMCBase` to support SONiC-on-BMC (`DEVICE_METADATA|bmc os = sonic`) on Switch-Host: - When BMC OS is `sonic`, `get_eeprom()` reads Model, PartNumber, and SerialNumber from the BMC remote STATE_DB (`EEPROM_INFO` TLV keys `0x21`–`0x23`) via `daemon_base.db_connect_remote()`. - Redfish session-managed APIs (`with_session_management`) reject calls when BMC OS is `sonic`. - OpenBMC path (`os = openbmc`, default) is unchanged. #### Motivation and Context Switch-Host must talk to both OpenBMC peers (Redfish) and SONiC-on-BMC peers (Redis over the USB/BMC link). After CONFIG_DB exposes the BMC OS type, the platform code needs a shared implementation in `sonic-platform-common` so vendor BMC classes can serve EEPROM without Redfish on SONiC BMC. #### How Has This Been Tested? ```bash pytest tests/bmc_base_test.py -k "eeprom" -v show platform bmc eeprom ``` #### Additional Information (Optional) Signed-off-by: Ben Levi <belevi@nvidia.com>
benle7
force-pushed
the
sonic_bmc_redis
branch
from
August 26, 2026 16:21
54a9f96 to
70d06d9
Compare
Collaborator
|
/azp run |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Dependent on: sonic-net/sonic-buildimage#29206
Description
Extend
BMCBaseto support SONiC-on-BMC (DEVICE_METADATA|bmc os = sonic) on Switch-Host:sonic,get_eeprom()reads Model, PartNumber, and SerialNumber from theBMC remote STATE_DB (
EEPROM_INFOTLV keys0x21–0x23) viadaemon_base.db_connect_remote().with_session_management) reject calls when BMC OS issonic.os = openbmc, default) is unchanged.Motivation and Context
Switch-Host must talk to both OpenBMC peers (Redfish) and SONiC-on-BMC peers (Redis over the
USB/BMC link). After CONFIG_DB exposes the BMC OS type, the platform code
needs a shared implementation in
sonic-platform-commonso vendor BMC classes can serve EEPROMwithout Redfish on SONiC BMC.
How Has This Been Tested?
pytest tests/bmc_base_test.py -k "eeprom" -v show platform bmc eepromAdditional Information (Optional)