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

Add command line support for thermal control #777

Merged
merged 8 commits into from
Jan 24, 2020
Merged

Add command line support for thermal control #777

merged 8 commits into from
Jan 24, 2020

Conversation

Junchao-Mellanox
Copy link
Collaborator

@Junchao-Mellanox Junchao-Mellanox commented Dec 27, 2019

- What I did

  1. Add a "show platform fanstatus" command to allow user fetch FAN status data.
  2. Add a "show platform temperature" command to allow user fetch thermal status data.

- How I did it

  1. Add a fanshow script to get FAN status from database and display it in command line
  2. Add a tempershow script to get thermal status from database and display it in command line
  3. Add two sub-commands under "platform" group

- How to verify it

  1. Issue "show platform fanstatus" in command line and see the output.
  2. Issue "show platform temperature" in command line and see the output.

- Previous command output (if the output of a command-line utility has changed)

- New command output (if the output of a command-line utility has changed)

show/main.py Outdated Show resolved Hide resolved
scripts/tempershow Outdated Show resolved Hide resolved
@Junchao-Mellanox Junchao-Mellanox marked this pull request as ready for review January 7, 2020 02:04
keboliu
keboliu previously approved these changes Jan 11, 2020
@jleveque
Copy link
Contributor

@Junchao-Mellanox: Please fix the new conflict.

@liat-grozovik
Copy link
Collaborator

@stepanblyschak please review as well

@lguohan lguohan merged commit c139a23 into sonic-net:master Jan 24, 2020
abdosi pushed a commit that referenced this pull request Feb 24, 2020
Add a "show platform fanstatus" command to allow user fetch FAN status data.
Add a "show platform temperature" command to allow user fetch thermal status data.
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.

6 participants