-
Notifications
You must be signed in to change notification settings - Fork 669
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[gendump] Add Support to dump BCM-DNX commands (#1813)
In generate_dump script, the function collect_broadcom tries to dump BCM XGS commands even in DNX switchs and error message is printed, hence generate_dump command fails. Modified the collect_broadcom function to check for BCM family and dump the DNX commands if the family is DNX. Signed-off-by: Sakthivadivu Saravanaraj sakthivadivu.saravanaraj@nokia.com What I did Modified the generate_dump script to dump the corresponding Broadcom diag shell commands based on Broadcom asic family. How I did it Modified the collect_broadcom function in generate_dump to get the Broadcom asic family by extracting the "version" command output as per Broadcom's suggestion and dump the Broadcom DNX commands if the Broadcom asic is DNX family. If it is not DNX family, dump the existing XGS commands. How to verify it Ran sonic cli command "sudo generate_dump " in DNX platform and XGS platform and verified that the tgz files are created with all the information dumped by generate_dump script. 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)
- Loading branch information
1 parent
0e31790
commit 53ff644
Showing
1 changed file
with
68 additions
and
22 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters