Make show int stat on supervisor trigger rexec#4529
Conversation
Signed-off-by: Javier-Tan <47554099+Javier-Tan@users.noreply.github.com>
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates show int stat behavior on chassis supervisors so it triggers rexec and returns consolidated interface status from all linecards, aligning with other supervisor “fan-out” show commands.
Changes:
- Detect supervisor context in
show interfaces statusand execute the same command remotely on all linecards viarexec all. - Print a user-facing message indicating the command is being run remotely.
- Exit with the remote execution’s return code.
|
/azp run |
liamkearney-msft
left a comment
There was a problem hiding this comment.
Looks good to me. Maybe we should think about putting this rexec logic / print into a common function to use here / for show ip bgp.
Then if we want to apply this to more commands we dont keep duplicating.
We can do this on next occurrence though, if you dont want to change that here in this PR
|
/azpw run |
|
Retrying failed(or canceled) jobs... |
|
No Azure DevOps builds found for #4529. |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Signed-off-by: Javier-Tan <47554099+Javier-Tan@users.noreply.github.com>
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Fixes sonic-net/sonic-buildimage#27222
What I did
Change
show int staton chassis supervisor to triggerrexec, to returnshow int statoutput from LCs (as happens when runningshow ip bgp ...on supervisors)How I did it
Add the same functionality under
statusas is underbgpfunctionsHow to verify it
Edited python file on device, to ensure it works
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)