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

[202012][submodule] Advance sonic-platform-common pointer #12551

Conversation

dprital
Copy link
Collaborator

@dprital dprital commented Oct 30, 2022

Signed-off-by: dprital drorp@nvidia.com

Why I did it

Update sonic-platform-common submodule pointer to include the following:

  • a2164f6 [202012] Fix issue: sff8436 does not support decoding power class 5~7 (#317)
  • 6722b4e Update the return of update_firmware for the image not exist case (#306)

How I did it

Advance sonic-platform-common pointer

How to verify it

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

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

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

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

Update sonic-platform-common submodule pointer to include the following:
* a2164f6 [202012] Fix issue: sff8436 does not support decoding power class 5~7 ([sonic-net#317](sonic-net/sonic-platform-common#317))
* 6722b4e Update the return of update_firmware for the image not exist case ([sonic-net#306](sonic-net/sonic-platform-common#306))

Signed-off-by: dprital <drorp@nvidia.com>
@dprital dprital marked this pull request as ready for review October 30, 2022 09:31
@liat-grozovik liat-grozovik merged commit 67b6541 into sonic-net:202012 Oct 30, 2022
@dprital dprital deleted the sub_module_202012_update_sonic-platform-common branch October 30, 2022 17:38
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.

2 participants