Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

[device/celestica]: Implement thermal base API based on the new platform API #3220

Merged
merged 2 commits into from
Aug 5, 2019
Merged

[device/celestica]: Implement thermal base API based on the new platform API #3220

merged 2 commits into from
Aug 5, 2019

Conversation

mudsut4ke
Copy link
Contributor

- What I did

  • Implement device base APIs
    • Thermal APIs
      • get_temperature()
      • get_high_threshold()
      • set_high_threshold()
    • Device APIs
      • get_name()
      • get_presence()
      • get_status()

- How I did it

  1. Update sensor configuration file follow hardware specification document
  2. Implement Thermal APIs
  3. Add Thermal object to chassis API

- How to verify it

Signed-off-by: Wirut Getbamrung wgetbumr@celestica.com

@jleveque
Copy link
Contributor

Retest vsimage please

@mudsut4ke
Copy link
Contributor Author

@jleveque Please merge this PR

@jleveque jleveque merged commit 1dd2688 into sonic-net:master Aug 5, 2019
@mudsut4ke mudsut4ke deleted the master-add-thermal-api branch August 6, 2019 07:48
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@jleveque jleveque jleveque approved these changes

Assignees
No one assigned
Labels
Enhancement ➕ Platform: Celestica
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mudsut4ke @jleveque