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.

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][Innovium] Added support for Aurora 615/715 #9335

Open
wants to merge 12 commits into
base: 202012
Choose a base branch
from

Conversation

andrewsapronov
Copy link
Contributor

@andrewsapronov andrewsapronov commented Nov 21, 2021

nba615-showplatform-211116.txt
nba715-showplatform-211116.txt
Signed-off-by: Andrew Sapronov andrew.sapronov@gmail.com

Why I did it

Suport Innovium based platforms Netberg Aurora 715/615
ASIC: Innovium Teralynx IVM 55300 (715) and 55200 (615)
Pors: 32x 100G and 48x 25G + 8x 100

How I did it

Added specification to device/netberg directory
Added platform/innovium/sonic-platform-modules-netberg contains kernel modules, scripts and sonic_platform packages.
Modified the platform/innovium/one-image.mk and platform/innovium/rule.mk to include Aurora 715/615 related ID and files.

How to verify it

Build sonic 202012 branch

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)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Andrew Sapronov <andrew.sapronov@gmail.com>
@lgtm-com
Copy link

lgtm-com bot commented Nov 21, 2021

This pull request introduces 25 alerts when merging 8ebe13c into 678100a - view on LGTM.com

new alerts:

  • 16 for Wrong number of arguments in a class instantiation
  • 3 for Unused import
  • 2 for Module imports itself
  • 2 for Unused local variable
  • 2 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Nov 23, 2021

This pull request introduces 23 alerts when merging ae59be0 into 38553f2 - view on LGTM.com

new alerts:

  • 16 for Wrong number of arguments in a class instantiation
  • 2 for Module imports itself
  • 2 for Unused local variable
  • 2 for 'import *' may pollute namespace
  • 1 for Unused import

Signed-off-by: Andrew Sapronov <andrew.sapronov@gmail.com>
@lgtm-com
Copy link

lgtm-com bot commented Nov 23, 2021

This pull request introduces 18 alerts when merging 8b9c066 into 38553f2 - view on LGTM.com

new alerts:

  • 16 for Wrong number of arguments in a class instantiation
  • 2 for Unused local variable

ArunSaravananBalachandran pushed a commit to ArunSaravananBalachandran/sonic-buildimage that referenced this pull request Nov 24, 2021
Why I did it
Update FRR 7.2.1 head. The following is a list of new commits.

5ae667a1f Merge pull request sonic-net#9335 from FRRouting/mergify/bp/stable/7.2/pr-9214
eb679e8a1 zebra: bugfix of error quit of zebra, due to no nexthop ACTIVE
80d2eaa98 Merge pull request sonic-net#8886 from FRRouting/mergify/bp/stable/7.2/pr-8876
1eeab2c1e lib: remove pure attribute from functions that modify memory
eb00dc4ec Merge pull request sonic-net#6944 from LabNConsulting/working/lb/7.2/valgrind-supp-libyang
b9d6d05bf bgpd: suppress new libyang_1.0 related loss reports
8c26a71eb Merge pull request sonic-net#6562 from ton31337/fix/configuration_for_labeled_unicast_in_place_7.2
386a1719c bgpd: Make sure network/aggregate-address commands lay down under labeled safi
b01c8bf28 Merge pull request sonic-net#6526 from ton31337/fix/set_ipv6_ll_if_global_zero_7.2
c382833e8 bgpd: Use IPv6 LL address as nexthop if global was set to ::/LL
99509b835 Merge pull request sonic-net#6395 from opensourcerouting/7.2/init-config-perms
7eef8f7b1 build: use configfile mode in init script
4cbe07705 Merge pull request sonic-net#6360 from opensourcerouting/7.2/fix-warnings
84bb11785 nhrpd: clean up SA warning
aac726476 nhrpd: be more careful with linked lists
3a4b6d654 debian: Fix spelling error
756c67c6c Merge pull request sonic-net#6284 from opensourcerouting/7.2/gcc-10
65a116a64 Merge pull request sonic-net#6354 from ton31337/fix/communities_bgpd_crash_7.2
f7a00fd67 bgpd: Check to ensure community attributes exist before freeing them
a960f99c2 vrrpd: fix build on Fedora Rawhide
d4caff99f babeld: GCC complaining about no return in non-void function
a014c27ae babeld: fix build on Fedora Rawhide
79ff55b5b bgpd: remove unused variable
ff343e588 pimd: Make frr able to be built by gcc 10
9a3cf1ba2 ldpd: remove multiple definitions of thread_master
a19515bfe ldpd: fix another linking issue with GCC-10
b4c8de38c tests: fix build with GCC 10
4f27e8c85 ldpd: Fix linking error on Fedora Rawhide with GCC 10

How I did it
Update FRR 7.2 pointer and create a tag frr-7.2.1-s4.
@andrewsapronov
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 9335 in repo Azure/sonic-buildimage

@andrewsapronov
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@andrewsapronov
Copy link
Contributor Author

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 9335 in repo Azure/sonic-buildimage

@andrewsapronov
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@andrewsapronov
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@lgtm-com
Copy link

lgtm-com bot commented May 20, 2022

This pull request introduces 18 alerts when merging bc55502 into cceef8e - view on LGTM.com

new alerts:

  • 16 for Wrong number of arguments in a class instantiation
  • 2 for Unused local variable

@lgtm-com
Copy link

lgtm-com bot commented May 25, 2022

This pull request introduces 17 alerts when merging e87d464 into 455d44e - view on LGTM.com

new alerts:

  • 16 for Wrong number of arguments in a class instantiation
  • 1 for Unused local variable

Signed-off-by: Andrew Sapronov <andrew.sapronov@gmail.com>
@lgtm-com
Copy link

lgtm-com bot commented May 26, 2022

This pull request introduces 16 alerts when merging f56a59e into a76899b - view on LGTM.com

new alerts:

  • 16 for Wrong number of arguments in a class instantiation

@lgtm-com
Copy link

lgtm-com bot commented May 26, 2022

This pull request introduces 16 alerts when merging 266ad90 into 9b84294 - view on LGTM.com

new alerts:

  • 16 for Wrong number of arguments in a class instantiation

@andrewsapronov
Copy link
Contributor Author

Hi,

We merged and test with latest 202012 and fixed all LGTM warnings related to our code.
@lguohan do we need to do something more to merge this PR?

@lgtm-com
Copy link

lgtm-com bot commented Jul 18, 2022

This pull request introduces 16 alerts when merging 2b215f9 into c5eb031 - view on LGTM.com

new alerts:

  • 16 for Wrong number of arguments in a class instantiation

@lgtm-com
Copy link

lgtm-com bot commented Oct 4, 2022

This pull request introduces 18 alerts when merging b8c67bd into a16843a - view on LGTM.com

new alerts:

  • 16 for Wrong number of arguments in a class instantiation
  • 2 for Comparison using is when operands support `__eq__`

@qiluo-msft qiluo-msft force-pushed the 202012 branch 2 times, most recently from 6fd3d83 to 06be005 Compare March 6, 2023 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants