-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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.
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
[Netberg][Barefoot] Added support for Aurora 610 #10579
[Netberg][Barefoot] Added support for Aurora 610 #10579
Conversation
Why I did it Pcied running by python 2. How I did it dropped python2 support and add python3 support for pcied in file docker-pmon.supervisord.conf.j2 How to verify it docker exec pmon supervisorctl status
Signed-off-by: Andrew Sapronov <andrew.sapronov@gmail.com>
Signed-off-by: Andrew Sapronov <andrew.sapronov@gmail.com>
This pull request introduces 32 alerts when merging b441d42 into 0191300 - view on LGTM.com new alerts:
|
Signed-off-by: Andrew Sapronov <andrew.sapronov@gmail.com>
This pull request introduces 14 alerts when merging 6c4a85f into 869ac1d - view on LGTM.com new alerts:
|
Signed-off-by: Andrew Sapronov <andrew.sapronov@gmail.com>
This pull request introduces 7 alerts when merging 892e9b5 into 869ac1d - view on LGTM.com new alerts:
|
Dear @sujinmkang @prgeor could you please review and merge this PR. |
@andrewsapronov Can you please check if the 7 new lgtm alerts need to be addressed? And also can you please provide the |
It's hard to say for me. All of the alerts connected with different numbers of __init__arguments across a broad vendors.
Yep, it have already linked in the first message - nba610-showplatform-master-220414.txt
Please check - nba610-pmon-daemons-sensors-master-220511.txt
We don't have this yet. |
…to nba610-initial-support
This pull request introduces 7 alerts when merging cf6bf3e into 937bf09 - view on LGTM.com new alerts:
|
Dear @sujinmkang do you need something else to approve this PR. We also check our kernel modules for the recent kernel version update - #10634. They are OK - no changes are needed. |
Signed-off-by: Andrew Sapronov <andrew.sapronov@gmail.com>
…to nba610-initial-support
This pull request introduces 7 alerts when merging 0fbb1c2 into ed99ce0 - view on LGTM.com new alerts:
|
@akokhan thanks for the review. @lguohan, @sujinmkang could you please merge the PR. |
nba610-showplatform-master-220414.txt
Why I did it
Support Intel Tofino based platforms Netberg Aurora 610
ASIC: Intel Tofino BFN-T10-032D-020
Pors: 48x 25G + 8x 100G
How I did it
Added specification to device/netberg directory
Added platform/barefoot/sonic-platform-modules-netberg contains kernel modules, scripts and sonic_platform packages.
Modified the platform/barefoot/one-image.mk and platform/barefoot/rule.mk to include Aurora 610 related ID and files.
How to verify it
Build SONiC
Install the image on the device and verify the related components are installed and shown correctly.
Which release branch to backport (provide reason below if selected)
Description for the changelog
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)