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.

Sign up for GitHub

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

Jump to bottom

[mellanox]: Fix problem with system EEPROM in "hw-mgmt" #1938

Merged
merged 1 commit into from
Aug 16, 2018
Merged

[mellanox]: Fix problem with system EEPROM in "hw-mgmt" #1938

merged 1 commit into from
Aug 16, 2018

Conversation

volodymyrsamotiy
Copy link
Collaborator

@volodymyrsamotiy volodymyrsamotiy commented Aug 16, 2018 •
edited
Loading

Signed-off-by: Volodymyr Samotiy volodymyrs@mellanox.com

- What I did
Fixed problem with system EEPROM in hw-mgmt
- How I did it
Apply patch to hw-mgmt sub-module in order to add valid modprobe config for the at24 module
(Also removed stop action for hw-managemet service since it might cause errors)
- How to verify it
Build an image, deploy to the switch and verify that decode-syseeprom returns valid output without any errors
- Description for the changelog
[mellanox]: Fix problem with system EEPROM in "hw-mgmt"

[mellanox]: Fix problem with system EEPROM in "hw-mgmt"
88e0b1c 
Signed-off-by: Volodymyr Samotiy <volodymyrs@mellanox.com>
@lguohan lguohan merged commit 986a857 into sonic-net:master Aug 16, 2018
@qiluo-msft
Copy link
Collaborator

removed stop action for hw-managemet service since it might cause errors

@volodymyrsamotiy Could you elaborate this? It is weird to remove stopping service statement.

@liuh-80 liuh-80 mentioned this pull request Nov 22, 2021
Merged
5 tasks
lguohan pushed a commit that referenced this pull request Nov 22, 2021
@liuh-80
[sonic-utilities] submodule update (#9342)
a5bf6fd 
Submodule update for sonic-utilities with following change:

ec9e5ee Backport [generate_dump] remove secrets from dump files #1886 to 202012 (#1938)
ce3b856 [fdbshow]: Handle FDB cleanup gracefully. (#1926)
1437bf2 [202012] Add DHCPv6 Relay counter and ipv6 helper CLI (#1917)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Bug 🐛 Platform: Mellanox
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@volodymyrsamotiy @qiluo-msft @lguohan @stcheng