Fix arubaOS-CX power-consumption changes logged #3143
Conversation
Exclude actual power consumption from output. Command "show environment power-consumption" has been introduced with ArubaOS-CX Long Support Release (LSR) version FL.10.13.xxx. This was not included in the previous LSR FL.10.10.xxx. Related issue is ytti#3142. Based on the work of @danpoltawski
Exclude actual power consumption from output. Command "show environment power-consumption" has been introduced with ArubaOS-CX Long Support Release (LSR) version FL.10.13.xxx. This was not included in the previous LSR FL.10.10.xxx. Related issue is ytti#3142. Based on the work of @danpoltawski
|
Hi @TerraTalpi , I tested out the changes in your PR, and i found that the changes DO work, but leave soemthing to be desired... it will still produce a "change" if any of the power numbers changes from 2 digits to 3 digits (ie if from one oxidixed run to another, instantaneous power goes from "99.50" to "103.50", then the output after that PR will go from this to this: also, it doesn't capture or strip total power consumption, so any changes there will still be captured |
fixes power consumptipon for up to a for digit number and keep table formatting. Starting with ArubaOS-CX Version FL.10.13.xxx
|
Dear maintainers, ready to merge from my side, I have vastly expanded support for multiple arubaOS-CX switch series and chassis switches. For details, see related issue with the deeper analysis of the outputs that may come and must be parsed. |
|
@alextremblay - does the PR now works for you? |
remove trailing white spaces of line "# match Total Power consumption:"
|
Hi @robertcheramy thank you for approving the CI/CD runs. I have corrected the trailing whitespaces error, sorry for that. Please run the tests again in order for all tests to pass. |
|
@alextremblay - I will merge this PR. If you miss something, this can be changed / added afterwards. |
Pre-Request Checklist
Description
Fix #3142. The ArubaOS-CX switches with the new Long Support Release (LSR) Version FL.10.13.xxxx introduce a new sub-command
show environment power-consumption. This displays current power consumption values for all switches in the stack. This is a fix, to exclude all actual power consumption values from the output.