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

[Bug fix] fix a syntax error which was introduced by last commit #71

Merged
merged 7 commits into from
Dec 10, 2019
Merged

[Bug fix] fix a syntax error which was introduced by last commit #71

merged 7 commits into from
Dec 10, 2019

Conversation

keboliu
Copy link
Collaborator

@keboliu keboliu commented Dec 10, 2019

Fix a typo which was introduced by #70 , this fix was verified, shall be OK.

@keboliu keboliu changed the title [Bug fix] fix a typo which introduce by last commit [Bug fix] fix a typo which was introduced by last commit Dec 10, 2019
@qiluo-msft qiluo-msft changed the title [Bug fix] fix a typo which was introduced by last commit [Bug fix] fix a syntax error which was introduced by last commit Dec 10, 2019
@qiluo-msft qiluo-msft merged commit 6f74dd3 into sonic-net:master Dec 10, 2019
lguohan pushed a commit to sonic-net/sonic-buildimage that referenced this pull request Dec 11, 2019
sonic-net/sonic-platform-common#71 [Bug fix] fix a syntax error which was introduced by last commit
sonic-net/sonic-platform-common#70 Fix EEPROM vendor extension TLV number counting issue
sonic-net/sonic-platform-common#62 Platform Driver Development Framework (PDDF): 1) Changes to psu base class (1.0 APIs) to support PDDF CLI utils, 2) Adding fan_base class to support PDDF fan CLI utils (comments incorporated)
sonic-net/sonic-platform-common#68 DeviceBase.get_name should raise NotImplementedError like other member
abdosi pushed a commit that referenced this pull request Jan 3, 2020
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-platform-common that referenced this pull request Oct 25, 2024
When a fan is under\over speed, status should change to 'Not OK' along with red LED color.

Signed-off-by: Shlomi Bitton <shlomibi@mellanox.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants