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

DellEMC S6100 : Platform2.0 API implementation [Module, Thermal] #3363

Merged
merged 5 commits into from
Sep 14, 2019
Merged

DellEMC S6100 : Platform2.0 API implementation [Module, Thermal] #3363

merged 5 commits into from
Sep 14, 2019

Conversation

ArunSaravananBalachandran
Copy link
Contributor

- What I did

  • Implement Module and Thermal platform2.0 API for DellEMC S6100 platform
  • Implement get_system_eeprom_info API

- How I did it

  • Added new files module.py and thermal.py for module and thermal platform2.0 API implementation
  • Made change in eeprom.py for implementing get_system_eeprom_info

- How to verify it

Wrote python script to load Chassis class and then call get_system_eeprom_info, Module and Thermal APIs accordingly.
UT Logs : UT_logs-module.txt, UT_logs-thermal.txt, UT_logs-eeprom.txt

- Description for the changelog

DellEMC S6100 : Platform2.0 API implementation [Module, Thermal]

- A picture of a cute animal (not mandatory but encouraged)

@sujinmkang
Copy link
Collaborator

retest vs please

@sujinmkang sujinmkang merged commit faa754f into sonic-net:master Sep 14, 2019
@ArunSaravananBalachandran ArunSaravananBalachandran deleted the platform2.0_s6100 branch September 17, 2019 03:15
mssonicbld added a commit that referenced this pull request Nov 23, 2024
…lly (#20905)

#### Why I did it
src/sonic-swss
```
* 1847195c - (HEAD -> master, origin/master, origin/HEAD) Add port FEC BER feature swss, HLD#1829 (#3363) (9 hours ago) [vincentpcng]
* 3d5ac540 - Create counter for the queue to which the host CPU traffic is sent when create_only_config_db_buffers is enabled (#3334) (10 hours ago) [Stephen Sun]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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.

3 participants