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

[sonic-platform-common] Update submodule #7868

Merged
merged 1 commit into from
Jun 18, 2021

Conversation

Junchao-Mellanox
Copy link
Collaborator

@Junchao-Mellanox Junchao-Mellanox commented Jun 15, 2021

7670b49 [sonic_platform/sfp_base] Add common definition for get SFP error status (#194)
1336598 [CI] sonic-config-engine now depends on SONiC YANG packages (#198)
f57fee4 Add to check pcie configuration revision to get the right configuration. (#195)
4e3a0a0 Fix typo for midplane APIs. (#196)
fc2e9e2 [eeprom_tlv_info] Optimize EEPROM data process by using visitor pattern (#193)

Why I did it

Update submodule for sonic-platform-common to include PR "Optimize EEPROM data process by using visitor pattern (#193)"

How I did it

Update submodule for sonic-platform-commo

How to verify it

Run build

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

  • 201811
  • 201911
  • 202006
  • 202012

Description for the changelog

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

@liat-grozovik
Copy link
Collaborator

/azp run

liat-grozovik
liat-grozovik previously approved these changes Jun 15, 2021
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liat-grozovik
Copy link
Collaborator

submodule update PR #7868 should be merged first.

@stephenxs
Copy link
Collaborator

Junchao, can you advance it to sonic-net/sonic-platform-common#198?
Thanks

@stephenxs
Copy link
Collaborator

sonic-net/sonic-platform-common#194 as well
Thanks

@liat-grozovik
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jleveque
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Junchao-Mellanox
Copy link
Collaborator Author

The job "Clean Workspace" failed, it seems not related to the change, could you please help check? @jleveque

@jleveque
Copy link
Contributor

The job "Clean Workspace" failed, it seems not related to the change, could you please help check? @jleveque

This is seen on other PRs, as well. Will try re-running.

@jleveque
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jleveque jleveque changed the title [submodule] Update submodule for sonic-platform-common [sonic-platform-common] Update submodule Jun 18, 2021
@jleveque jleveque merged commit be5fc77 into sonic-net:master Jun 18, 2021
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
7670b49 [sonic_platform/sfp_base] Add common definition for get SFP error status (sonic-net#194)
1336598 [CI] sonic-config-engine now depends on SONiC YANG packages (sonic-net#198)
f57fee4 Add to check pcie configuration revision to get the right configuration. (sonic-net#195)
4e3a0a0 Fix typo for midplane APIs. (sonic-net#196)
fc2e9e2 [eeprom_tlv_info] Optimize EEPROM data process by using visitor pattern (sonic-net#193)
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.

None yet

4 participants