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

[Seastone] DX010 platform switch to sfp-refactor based sfp impl #13972

Merged
merged 1 commit into from
Mar 27, 2023

Conversation

qnos
Copy link
Contributor

@qnos qnos commented Feb 24, 2023

Why I did it

sonic-sfp based sfp impl would be deprecated in future, change to sfp-refactor based implementation.

How I did it

Use the new sfp-refactor based sfp implementation.

How to verify it

Manual test sfp platform api or run sfp platform test cases.

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)

Why I did it
sonic-sfp based sfp impl would be deprecated in future, change to sfp-refactor based implementation.

How I did it
Use the new sfp-refactor based sfp implementation for seastone.

How to verify it
Manual test sfp platform api or run sfp platform test cases.
@qnos qnos force-pushed the dx010-switch-to-sfp-refactor-impl branch from 5de98e4 to 809f2af Compare February 27, 2023 23:25
@qnos
Copy link
Contributor Author

qnos commented Feb 28, 2023

@assrinivasan Please help to review. Thanks

@assrinivasan
Copy link
Contributor

@mihirpat1 please check.

@prgeor
Copy link
Contributor

prgeor commented Feb 28, 2023

@lguohan please help merge

@lguohan lguohan merged commit f550c86 into sonic-net:master Mar 27, 2023
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Mar 31, 2023
…c-net#13972)

Why I did it
sonic-sfp based sfp impl would be deprecated in future, change to sfp-refactor based implementation.

How I did it
Use the new sfp-refactor based sfp implementation for seastone.

How to verify it
Manual test sfp platform api or run sfp platform test cases.
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202205: #14485

mssonicbld pushed a commit that referenced this pull request Apr 3, 2023
Why I did it
sonic-sfp based sfp impl would be deprecated in future, change to sfp-refactor based implementation.

How I did it
Use the new sfp-refactor based sfp implementation for seastone.

How to verify it
Manual test sfp platform api or run sfp platform test cases.
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.

7 participants