Add support for Accton wedge100bf_32qs platform to SONiC master #9257
Conversation
This pull request is based on wedge100bf_32x. The components on the mainboard are the same as wedge100bf_32x, except for tofino 32Q and COMe models, so it refers to wedge100bf_32x to create new model: wedge100bf_32qs. Signed-off-by: alvin_feng <[email protected]>
|
This pull request introduces 2 alerts when merging 8ad0e14 into db0b789 - view on LGTM.com new alerts:
|
Signed-off-by: alvin_feng <[email protected]>
platform/barefoot/sonic-platform-modules-accton/wedge100bf-32qs/sonic_platform/__init__.py
Outdated
Show resolved
Hide resolved
…sonic_platform Signed-off-by: alvin_feng <[email protected]>
|
@lguohan |
device/accton/x86_64-accton_wedge100bf_32qs-r0/montara/switch-sai.conf
Outdated
Show resolved
Hide resolved
|
@alvinkaiwei , do we need a separate PR for 202012? Can't you just update this PR's description with an original SONiC PR template and in "Which release branch to backport" section mark 202012 ? Please see how it's done in "#9465" |
@akokhan,The definition of the platform.json file is different between 202012 and master. |
Signed-off-by: alvin_feng <[email protected]>
We have discussed this issue with the intel team, so we will close PR#9065 and mark it to be cherry-picked into 202012/202106/202111. |
|
@akokhan May you please help review this PR again? EC customer is waiting for that. |
|
@lguohan Please help to check if this modification can be merged into SONiC master. Thanks! |
|
Jus FYI |
…rect generating BUFFER_QUEUE values in DB. Signed-off-by: alvin_feng <[email protected]>
alvinkaiwei
requested review from
prgeor,
sujinmkang and
vdahiya12
as code owners
@akokhan I have modified platform.json to avoid platform TCs issues and changes for correct generating BUFFER_QUEUE values in DB. |
|
@akokhan Alvin already add all the modifications you suggested and passed the checks. May you please help merge it to master and branches, 202012, 202106, and 202111. |
Why I did it
This pull request is based on wedge100bf_32x.
The components on the mainboard are the same as wedge100bf_32x, except for tofino 32Q and COMe models, so it refers to wedge100bf_32x to create new model: wedge100bf_32qs.
Signed-off-by: alvin_feng [email protected]
How I did it
It refers to wedge100bf_32x to create new model: wedge100bf_32qs.
How to verify it
Verify on the 32qs device.
show interface status
sfputil show eeprom
sfputil show presence
show platform syseeprom
show platform psustatus
Which release branch to backport (provide reason below if selected)
Description for the changelog
A picture of a cute animal (not mandatory but encouraged)