[Mellanox] Read EEPROM data from DB if possible#7808
Merged
liat-grozovik merged 1 commit intosonic-net:masterfrom Jun 20, 2021
Merged
[Mellanox] Read EEPROM data from DB if possible#7808liat-grozovik merged 1 commit intosonic-net:masterfrom
liat-grozovik merged 1 commit intosonic-net:masterfrom
Conversation
|
This pull request introduces 1 alert and fixes 4 when merging ba8e9d0 into b2a2cf0 - view on LGTM.com new alerts:
fixed alerts:
|
jleveque
approved these changes
Jun 11, 2021
Collaborator
Author
|
PR for updating submodule pointer: #7868 |
liat-grozovik
approved these changes
Jun 15, 2021
Collaborator
|
This commit could not be cleanly cherry-pick to 202012. Please submit another PR |
carl-nokia
pushed a commit
to carl-nokia/sonic-buildimage
that referenced
this pull request
Aug 7, 2021
- Why I did it Remove EEPROM cache file and use DB instead - How I did it Read EEPROM data from DB if possible If data is not ready in DB, read from hardware using a visitor pattern - How to verify it Manual test and regression
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
4 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.
Depends on sonic-net/sonic-platform-common#193
Why I did it
Remove EEPROM cache file and use DB instead
How I did it
How to verify it
Manual test and regression
Which release branch to backport (provide reason below if selected)
Description for the changelog
A picture of a cute animal (not mandatory but encouraged)