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] Updated platform APIs impl #9309

Closed
wants to merge 3 commits into from

Conversation

akokhan
Copy link
Contributor

@akokhan akokhan commented Nov 18, 2021

Signed-off-by: Andriy Kokhan andriyx.kokhan@intel.com

Why I did it

Added missing implementation for BFN platform APIs

How I did it

How to verify it

Run sonic-mgmt platform TCs

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

@akokhan akokhan force-pushed the bfn_platform_api_update branch 4 times, most recently from ab5a679 to e1285e4 Compare November 23, 2021 09:21
@akokhan
Copy link
Contributor Author

akokhan commented Nov 24, 2021

@qiluo-msft , @lguohan , please approve and merge

lguohan
lguohan previously approved these changes Nov 26, 2021
@lguohan
Copy link
Collaborator

lguohan commented Nov 26, 2021

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@akokhan akokhan force-pushed the bfn_platform_api_update branch 8 times, most recently from 5ff2471 to 40c3f00 Compare November 30, 2021 18:54
Andriy Kokhan added 2 commits November 30, 2021 11:10
Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com>
@akokhan
Copy link
Contributor Author

akokhan commented Dec 1, 2021

@lguohan , please merge.

@akokhan
Copy link
Contributor Author

akokhan commented Dec 2, 2021

@lguohan , this PR extends platform API implementation for BFN platform. Please approve and merge.

@lgtm-com
Copy link

lgtm-com bot commented Dec 3, 2021

This pull request introduces 2 alerts when merging 54831ee into d041e60 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for Variable defined multiple times

@akokhan akokhan requested a review from lguohan December 4, 2021 07:39
@akokhan
Copy link
Contributor Author

akokhan commented Dec 6, 2021

@lguohan , please approve and merge

@akokhan akokhan closed this Dec 15, 2021
@akokhan
Copy link
Contributor Author

akokhan commented Dec 15, 2021

The following PR includes these changes:
#9540

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