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

[Alphanetworks] Add new platform SNJ60D0-320F #8780

Merged
merged 3 commits into from
Oct 22, 2021

Conversation

juntseng62
Copy link
Contributor

@juntseng62 juntseng62 commented Sep 16, 2021

Why I did it

Added Support for Alphanetworks SNJ60D0-320F platform

How I did it

Implemented the support for Alphanetworks SNJ60D0-320F platform

Platform: x86_64-alphanetworks_snj60d0_320f-r0
HwSKU: Alphanetworks-SNJ60D0-320F
ASIC: broadcom
ASIC Count: 1

How to verify it

Verified the show command outputs

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

[Alphanetworks] Add new platform SNJ60D0-320F

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

Signed-off-by: juntseng62 juntseng62@gmail.com
@juntseng62 juntseng62 requested a review from lguohan as a code owner September 16, 2021 11:13
@lgtm-com
Copy link

lgtm-com bot commented Sep 16, 2021

This pull request introduces 14 alerts when merging 59ae7b0 into 83e7876 - view on LGTM.com

new alerts:

  • 11 for Wrong number of arguments in a class instantiation
  • 3 for 'import *' may pollute namespace

@juntseng62
Copy link
Contributor Author

@sujinmkang @prgeor - Can you please review and approve the changes ?
Thanks

@sujinmkang
Copy link
Collaborator

@juntseng62 Can you attach the show version and show platform {} commands output ?

@juntseng62
Copy link
Contributor Author

juntseng62 commented Sep 27, 2021

@sujinmkang Sure. Please see attached file. Thanks.

SNJ60D0-320F_SONC_test.log

Signed-off-by: juntseng62 juntseng62@gmail.com
@lgtm-com
Copy link

lgtm-com bot commented Oct 8, 2021

This pull request introduces 14 alerts when merging 33908b5 into 3855ce2 - view on LGTM.com

new alerts:

  • 11 for Wrong number of arguments in a class instantiation
  • 3 for 'import *' may pollute namespace

@juntseng62
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@juntseng62
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@juntseng62
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@juntseng62
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

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

@juntseng62
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@juntseng62
Copy link
Contributor Author

@sujinmkang Can you please review and approve the changes ? Thanks.

Signed-off-by: juntseng62 juntseng62@gmail.com
@lgtm-com
Copy link

lgtm-com bot commented Oct 15, 2021

This pull request introduces 14 alerts when merging a76687f into c374705 - view on LGTM.com

new alerts:

  • 11 for Wrong number of arguments in a class instantiation
  • 3 for 'import *' may pollute namespace

@juntseng62
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@juntseng62
Copy link
Contributor Author

@sujinmkang Can you please review and approve the changes ? Thanks.

@sujinmkang sujinmkang merged commit 9632178 into sonic-net:master Oct 22, 2021
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.

3 participants