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

[submodule] Advance sonic-platform-common pointer #11965

Conversation

dprital
Copy link
Collaborator

@dprital dprital commented Sep 4, 2022

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

Why I did it

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

  • Read CMIS data path state duration (#312)
  • Remove shell=True (#313)
  • [credo][ycable] remove mux-toggle inprogress flags for some API's (#311)
  • [Cdb fw upgrade (#308)
  • [Credo][Ycable] enhancement and error exception for some APIs (#303)
  • [ycable] add definitions of some new API's for Y-Cable infrastructure (#301)
  • Install libyang to azure pipeline (#310)
  • Update the return of update_firmware for the image not exist case (#306)
  • [CMIS] 'get_transceiver_info' should return 'None' when CMIS cable EEPROM is not ready (#305)
  • uplift code coverage 80% (#307)
  • [sonic-pcie] Add UT for pcie_common.py (#293)
  • [CMIS] Catch Exception to avoid CMIS code crash (#299)
  • [Credo][Ycable] fix incorrect uart statistics (#296)
  • Add PSU input voltage and input current (#295)

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

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:
* [CMIS] 'get_transceiver_info' should return 'None' when CMIS cable EEPROM is not ready  ([sonic-net#305](sonic-net/sonic-platform-common#305))
* uplift code coverage 80% ([sonic-net#307](sonic-net/sonic-platform-common#307))
* [sonic-pcie] Add UT for pcie_common.py ([sonic-net#293](sonic-net/sonic-platform-common#293))
* [CMIS] Catch Exception to avoid CMIS code crash ([sonic-net#299](sonic-net/sonic-platform-common#299))
* [Credo][Ycable] fix incorrect uart statistics ([sonic-net#296](sonic-net/sonic-platform-common#296))
* Add PSU input voltage and input current ([sonic-net#295](sonic-net/sonic-platform-common#295))

Signed-off-by: dprital <drorp@nvidia.com>
@dprital
Copy link
Collaborator Author

dprital commented Sep 20, 2022

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@dprital
Copy link
Collaborator Author

dprital commented Sep 20, 2022

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@dprital
Copy link
Collaborator Author

dprital commented Sep 24, 2022

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 11965 in repo sonic-net/sonic-buildimage

@dprital
Copy link
Collaborator Author

dprital commented Sep 25, 2022

/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).

@bratashX
Copy link
Contributor

@dprital @mssonicbld Could you please update PR and restart CI?

@dprital
Copy link
Collaborator Author

dprital commented Oct 19, 2022

@dprital @mssonicbld Could you please update PR and restart CI?

@bratashX - Done

@liat-grozovik liat-grozovik merged commit 5a9a25e into sonic-net:master Oct 20, 2022
@dprital dprital deleted the submodule_master_update_sonic-platform-common branch October 20, 2022 11:13
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.

4 participants