Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[show][platform summary] Add chassis type in the platform summary #1922

Merged
merged 2 commits into from
Nov 15, 2021
Merged

[show][platform summary] Add chassis type in the platform summary #1922

merged 2 commits into from
Nov 15, 2021

Conversation

smaheshm
Copy link
Contributor

@smaheshm smaheshm commented Nov 10, 2021 •
edited
Loading

What I did

Add chassis type in the platform summary on chassis platforms.

How I did it

Get the switch type from platform_info() API.

How to verify it

admin@str2-chasssis-lc3-1:~$ show platform summary
Platform: chassis_lc
HwSKU: chassis
ASIC: broadcom
ASIC Count: 1
Serial Number: xyz
Model Number: chassis-LC
Hardware Revision: N/A
Switch Type: voq

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)

@smaheshm
Copy link
Contributor Author

commit after sonic-net/sonic-buildimage#9227

@smaheshm
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@smaheshm
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@smaheshm smaheshm mentioned this pull request Nov 11, 2021
Merged
4 tasks
@smaheshm
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@smaheshm
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@smaheshm
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@smaheshm smaheshm merged commit efa2ff6 into sonic-net:master Nov 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@judyjoseph judyjoseph judyjoseph approved these changes

@abdosi abdosi Awaiting requested review from abdosi

Assignees
No one assigned
Labels
Chassis show
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@smaheshm @judyjoseph