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

[BFN] [202111] Updated platform plugins #10422

Merged
merged 7 commits into from
Apr 14, 2022

Conversation

bratashX
Copy link
Contributor

Signed-off-by: Petro Bratash petrox.bratash@intel.com

Why I did it

Have a lot of missing BFN platform APIs implementation in SONiC 202111

How I did it

Ported BFN platform plugins from master branch.

How to verify it

Run sonic-mgmt platform TCs

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Link to config_db schema for YANG module changes

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

@bratashX bratashX requested a review from lguohan as a code owner March 31, 2022 22:42
@lgtm-com
Copy link

lgtm-com bot commented Mar 31, 2022

This pull request introduces 1 alert when merging 96d826b into 59af63f - view on LGTM.com

new alerts:

  • 1 for Wrong number of arguments in a class instantiation

@lgtm-com
Copy link

lgtm-com bot commented Apr 4, 2022

This pull request introduces 1 alert when merging 71af12e into 3522ac6 - view on LGTM.com

new alerts:

  • 1 for Wrong number of arguments in a class instantiation

@lgtm-com
Copy link

lgtm-com bot commented Apr 4, 2022

This pull request introduces 1 alert when merging be348662bd25a0d844f690b0f593b5468dfa4877 into 3522ac6 - view on LGTM.com

new alerts:

  • 1 for Wrong number of arguments in a class instantiation

@bratashX bratashX force-pushed the port_platform_to_202111 branch from be34866 to 5218546 Compare April 7, 2022 20:35
Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com>
@lgtm-com
Copy link

lgtm-com bot commented Apr 7, 2022

This pull request introduces 1 alert when merging 5218546 into 0acab75 - view on LGTM.com

new alerts:

  • 1 for Wrong number of arguments in a class instantiation

@bratashX
Copy link
Contributor Author

@akokhan @msosyak @SavchukRomanLv FYI

@lgtm-com
Copy link

lgtm-com bot commented Apr 11, 2022

This pull request introduces 1 alert when merging bd6a86b into 5809e7c - view on LGTM.com

new alerts:

  • 1 for Wrong number of arguments in a class instantiation

Updated SDE packages for bfn platform

- introduced X6 profile
- fixes for drop counters
- fixes for platform part
@bratashX
Copy link
Contributor Author

This pull request introduces 1 alert when merging bd6a86b into 5809e7c - view on LGTM.com

new alerts:

1 for Wrong number of arguments in a class instantiation

False-negative, opened issue: github/codeql#3617

@akokhan
Copy link
Contributor

akokhan commented Apr 13, 2022

@lguohan , this PR contains back porting of multiple BFN fixes including platform plugins updates from SONiC master. Please approve and merge.

@bratashX
Copy link
Contributor Author

@prgeor @sujinmkang, please review and merge

@prgeor prgeor merged commit 1af932a into sonic-net:202111 Apr 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants