Skip to content

Conversation

@yuazhe
Copy link
Contributor

@yuazhe yuazhe commented Dec 8, 2025

What I did

de-escalate the message when no psu had been detected at all from error to more moderate info.

How I did it

simply change the print output and remove the redundance ones

How to verify it

UT as well as manual test

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

Error: Failed to get the number of PSUs
Error: Failed to get PSU status
Error: failed to get PSU status from state DB

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

PSU not detected

@yuazhe yuazhe requested a review from stephenxs December 8, 2025 10:25
@mssonicbld
Copy link
Collaborator

/azp run

@yuazhe yuazhe requested a review from keboliu December 8, 2025 10:25
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yuazhe yuazhe force-pushed the master_fix_psushow branch from df76afc to db2cb49 Compare December 8, 2025 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants