sonic-utilities icon indicating copy to clipboard operation
sonic-utilities copied to clipboard

[psushow & psuutil] Support PSU power threshold checking

Open stephenxs opened this issue 3 years ago • 1 comments

What I did

Support PSU power threshold checking

  1. Introduce a new state for the PSU, indicating whether a PSU power threshold is exceeded
  2. For psuutil, expose the power thresholds
  3. Add HW Rev to psuutil

Signed-off-by: Stephen Sun [email protected]

How I did it

How to verify it

Unit and manual test

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)

stephenxs avatar Aug 23 '22 01:08 stephenxs

This pull request introduces 2 alerts when merging f5165b31148c82e0ecfd9e4aee901149551a3c1c into e14f679148159d7dc3f10fa2662fb8f7b202c3fc - view on LGTM.com

new alerts:

  • 2 for Testing equality to None

lgtm-com[bot] avatar Aug 23 '22 01:08 lgtm-com[bot]

@stephenxs could you resolve merge conflicts?

prgeor avatar Nov 18 '22 23:11 prgeor

@stephenxs Please populate these sections of the PR template:-

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)

prgeor avatar Nov 18 '22 23:11 prgeor

@stephenxs Please populate these sections of the PR template:-

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)

Done

stephenxs avatar Nov 19 '22 00:11 stephenxs

@stephenxs could you resolve merge conflicts?

Done by rebasing it.

stephenxs avatar Nov 19 '22 00:11 stephenxs

psutest.txt @prgeor a test log. all PSU test passed

stephenxs avatar Nov 21 '22 07:11 stephenxs