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

[platform-common] submodule update #13269

Merged
merged 1 commit into from
Jan 8, 2023

Conversation

dmytroxIntel
Copy link
Contributor

@dmytroxIntel dmytroxIntel commented Jan 5, 2023

sonic-platform-common:
9df998b (HEAD -> master, origin/master, origin/HEAD) Don't read AUX_MON_TYPE if memory model is flat (#339)
676b329 Removing null characters while decoding from syseeprom (#338)

Why I did it

Need to merge xcvr fix in cmis implementation from sonic-platform-common submodule

How I did it

How to verify it

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

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

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)

@dmytroxIntel
Copy link
Contributor Author

@lguohan , @prgeor , @StormLiangMS please merge this submodule update

@dmytroxIntel
Copy link
Contributor Author

@kperumalbfn , please merge this submodule update

@kperumalbfn
Copy link
Contributor

@dmytroxIntel Is this change required in 202211 branch?

@prgeor
Copy link
Contributor

prgeor commented Jan 7, 2023

@dmytroxIntel this commit is already available in the master. please remove:

676b329efd1b8cbe30b6e874308cd1ccf69466f8 Removing null characters while decoding from syseeprom (https://github.com/sonic-net/sonic-platform-common/pull/338)

@lguohan lguohan merged commit 95c148d into sonic-net:master Jan 8, 2023
@lguohan
Copy link
Collaborator

lguohan commented Jan 8, 2023

@prgeor , if it is already included, then the actuall commit does not include the commit. the pr is fine, just description is confusing.

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