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

[platform]add platfrom snh60a0-320fv2 & snh60b0-640f #1750

Closed
wants to merge 1 commit into from
Closed

[platform]add platfrom snh60a0-320fv2 & snh60b0-640f #1750

wants to merge 1 commit into from

Conversation

juntseng62
Copy link
Contributor

Add platform support for snh60b0-640f & snh60a0-320fv2 devices

ASIC Vendor: Broadcom
. Swich ASIC:
1.Tomhawk2 chip: for snh60b0-640f device
2.Tomhawk chip: for snh60a0-320fv2 device

. Port Config:
1.Tomhawk2 chip: for snh60b0-640f device: 64x100G
2.Tomhawk chip: for snh60a0-320fv2 device 32x100G

. What I did
1. Add new platform snh60b0-640f
2. Add new platform snh60a0-320fv2

. How I did it
Provide device and plarform related files.
. How to verify it
psuutil status
sfputil show presence
sfputil show eeprom
decode-syseeprom
sensors
fan policy check

. Description for the changelog

@msftclas
Copy link

msftclas commented May 30, 2018 •
edited
Loading

CLA assistant check
All CLA requirements met.

@lguohan
Copy link
Collaborator

lguohan commented Aug 12, 2018

@juntseng62, can you resolve conflict?

lguohan
lguohan requested changes Sep 1, 2018
View reviewed changes
Copy link
Collaborator

@lguohan lguohan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you resolve the conflict?

@juntseng62
Copy link
Contributor Author

Hi @lguohan ,
Thanks.
I think the Conficting file (.gitmodules) is because other vendor provides new platform at the same time.
Can I direct modify the conflicting file ".gitmodules" (To integrate our new added platform to latest .gitmodule file) or need to resend PR that include our new platform in latest SONiC source code ?
Please help to comment it.

/Jun

@juntseng62
Copy link
Contributor Author

Hi @lguohan ,
I will resent PR again

@juntseng62 juntseng62 closed this Sep 17, 2020
vivekrnv added a commit to vivekrnv/sonic-buildimage that referenced this pull request Sep 13, 2021
@vivekrnv
Update submodule sonic-utilities
4ad2756 
Signed-off-by: Vivek Reddy Karri <vkarri@nvidia.com>

2416175 [config reload] Fix config reload failure due to sonic.target job cancellation (sonic-net#1814)
2b12aad [portstat, intfstat] added rates and utilization (sonic-net#1750)
26e700a [debug dump util] Techsupport addition (sonic-net#1669)
9f2326e [debug dump util] Base Skeleton and Click Class added (sonic-net#1668)
@Junchao-Mellanox Junchao-Mellanox mentioned this pull request Sep 14, 2021
Merged
5 tasks
judyjoseph added a commit that referenced this pull request Sep 14, 2021
@judyjoseph
Update sonic-swss, sonic-utilities submodules
d283549 
sonic-swss commits

5fbd113 [Flex-counters] Fix the delay of flex counters flow to prevent infinite loop (#1899)
dc685b6 [portsorch] Add an extra check before setting oper speed to APPL_DB (#1885)
ceef728 Update port_rates & rif_rates lua scripts to convert poll_interval to MS (#1855)

sonic-utilities commits

3160753 [ci]: Support PR coverage (#1806)
3316fdb fix wrong code indent in sfputil (#1808)
c33e3a8 [config reload] Fix config reload failure due to sonic.target job cancellation (#1814)
4f7e107 [portconfig] Validate duplicate speed value and interface type value (#1745)
59817e2 [warm-reboot] Add new preboot health check: verify database integrity (#1785)
bf2ff3c [portstat, intfstat] added rates and utilization (#1750)
3bf962c [show][platform] Revise chassis info fallback to only fall back on pmon crash (#1751)
qiluo-msft pushed a commit that referenced this pull request Sep 28, 2021
@Junchao-Mellanox
[submodule] Update submodule for sonic-utilities (#8752)
c770375 
0d538d3 [ci]: Support code diff coverage (#1834)
48887d1 [config] support for configuring muxcable to standby mode of operation (#1837)
2088a9a Provide support to install platform extensions (#1578)
c97fe54 Add check_db_integrity script to setup.py (#1828)
c0b9917 [debug dump util] COPP Module Added (#1670)
826311c [techsupport] Removed interactive option for docker commands and Improved Error Reporting (#1723)
ce11545 [config reload] Removed job-mode for sonic.target restart (#1820)
f76f672 [fdbshow]: Fix typo in comment (#1809)
17208a0 [ci]: Support PR coverage (#1806)
c2c2354 fix wrong code indent in sfputil (#1808)
47a9a0f [portconfig] Validate duplicate speed value and interface type value (#1745)
f1086ee [sonic_installer]Add --skip-platform-check option for sonic_installer when image mismatch (#1791)
c007d65 [warm-reboot] Add new preboot health check: verify database integrity (#1785)
41e31e8 Fix PatchApplier init order (#1762)
2416175 [config reload] Fix config reload failure due to sonic.target job cancellation (#1814)
2b12aad [portstat, intfstat] added rates and utilization (#1750)
26e700a [debug dump util] Techsupport addition (#1669)
9f2326e [debug dump util] Base Skeleton and Click Class added (#1668)
taras-keryk pushed a commit to taras-keryk/sonic-buildimage that referenced this pull request Apr 28, 2022
@vadymhlushko-mlnx
[portstat, intfstat] added rates and utilization (sonic-net#1750)
2b12aad 
#### What I did

Depends on [sonic-swss-common/pull/330](sonic-net/sonic-swss-common#330)

According to [HLD](https://github.com/Azure/SONiC/blob/master/doc/rates-and-utilization/Rates_and_utilization_HLD.md) added calculation of rates and utilization columns to the `portstat` and `intfstat` scripts output

#### How I did it

Modified the `portstat` and `intfstat` scripts

#### How to verify it

Added UT

#### Previous command output (if the output of a command-line utility has changed)

```
admin@sonic:~$ show int count
      IFACE    STATE    RX_OK    RX_BPS    RX_UTIL    RX_ERR    RX_DRP    RX_OVR    TX_OK    TX_BPS    TX_UTIL    TX_ERR    TX_DRP    TX_OVR
-----------  -------  -------  --------  ---------  --------  --------  --------  -------  --------  ---------  --------  --------  --------
  Ethernet0        X        0       N/A        N/A         0         0       N/A        0       N/A        N/A         0         0       N/A
  Ethernet2        U      287       N/A        N/A         0         0       N/A      818       N/A        N/A         0         0       N/A
  Ethernet4        U      380       N/A        N/A         0         0       N/A      858       N/A        N/A         0         0       N/A
  Ethernet6        U      286       N/A        N/A         0         0       N/A      850       N/A        N/A         0         0       N/A
```

#### New command output (if the output of a command-line utility has changed)

```
admin@sonic:~$ show int count
      IFACE    STATE    RX_OK      RX_BPS    RX_UTIL    RX_ERR    RX_DRP    RX_OVR    TX_OK      TX_BPS    TX_UTIL    TX_ERR    TX_DRP    TX_OVR
-----------  -------  -------  ----------  ---------  --------  --------  --------  -------  ----------  ---------  --------  --------  --------
  Ethernet0        X        0    0.00 B/s      0.00%         0         0       N/A        0    0.00 B/s      0.00%         0         0       N/A
  Ethernet2        U        0    0.00 B/s      0.00%         0         0       N/A        0    0.00 B/s      0.00%         0         0       N/A
  Ethernet4        U        0    0.00 B/s      0.00%         0         0       N/A        0    0.00 B/s      0.00%         0         0       N/A
  Ethernet6        U        0    0.00 B/s      0.00%         0         0       N/A        0    0.00 B/s      0.00%         0         0       N/A
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lguohan lguohan lguohan requested changes

Assignees
No one assigned
Labels
Enhancement ➕ Platform: Alpha
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@juntseng62 @msftclas @lguohan @stcheng