Skip to content

[action] [PR:4542] Broadcom show techsupport comamnds update#347

Merged
r12f merged 1 commit into
Azure:202412from
mssonicbld:cherry/msft-202412/4542
May 26, 2026
Merged

[action] [PR:4542] Broadcom show techsupport comamnds update#347
r12f merged 1 commit into
Azure:202412from
mssonicbld:cherry/msft-202412/4542

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

Update Broadcom show techsupport commands for TH5 chipset.

What I did

Update Broadcom commands in collect_broadcom method.

How I did it

Run Broadcom commands based on the chipset type.

How to verify it

Run show techsupport on TH5 DUT and non-TH5 DUT. Verify the commands output in the dump package has no errors.

Signed-off-by: Sonic Build Admin sonicbld@microsoft.com

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)

Update Broadcom show techsupport commands for TH5 chipset.

<!--
    Please make sure you've read and understood our contributing guidelines:
    https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md

    ** Make sure all your commits include a signature generated with `git commit -s` **

    If this is a bug fix, make sure your description includes "closes #xxxx",
    "fixes #xxxx" or "resolves #xxxx" so that GitHub automatically closes the related
    issue when the PR is merged.

    If you are adding/modifying/removing any command or utility script, please also
    make sure to add/modify/remove any unit tests from the tests
    directory as appropriate.

    If you are modifying or removing an existing 'show', 'config' or 'sonic-clear'
    subcommand, or you are adding a new subcommand, please make sure you also
    update the Command Line Reference Guide (doc/Command-Reference.md) to reflect
    your changes.

    Please provide the following information:
-->

#### What I did
Update Broadcom commands in collect_broadcom method.

#### How I did it
Run Broadcom commands based on the chipset type.

#### How to verify it
Run show techsupport on TH5 DUT and non-TH5 DUT.  Verify the commands output in the dump package has no errors.

Signed-off-by: Sonic Build Admin <sonicbld@microsoft.com>

#### 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)
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: sonic-net/sonic-utilities#4542

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@r12f r12f merged commit 0ff9400 into Azure:202412 May 26, 2026
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants