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

[Mellanox] Fix incomplete platform name for MSN4600C in sfputil plugin #7259

Merged
merged 1 commit into from
Apr 8, 2021

Conversation

keboliu
Copy link
Collaborator

@keboliu keboliu commented Apr 8, 2021

Signed-off-by: Kebo Liu kebol@nvidia.com

Why I did it

Fixes #7162
The platform name for MSN4600C in sfputil pliugin is not complete: "x86_64-mlnx_msn4600c" -> "x86_64-mlnx_msn4600c-r0"

How I did it

Correct the platform name

How to verify it

run "sfputil show presence" command

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)

…4600c-r0"

Signed-off-by: Kebo Liu <kebol@nvidia.com>
@keboliu keboliu requested a review from jleveque as a code owner April 8, 2021 10:27
@liat-grozovik liat-grozovik changed the title [Nvidia/Mellanox] Fix incomplete platform name for MSN4600C in plugin [Mellanox] Fix incomplete platform name for MSN4600C in plugin Apr 8, 2021
@liat-grozovik liat-grozovik changed the title [Mellanox] Fix incomplete platform name for MSN4600C in plugin [Mellanox] Fix incomplete platform name for MSN4600C in sfputil plugin Apr 8, 2021
@jleveque jleveque merged commit ef98890 into sonic-net:master Apr 8, 2021
yxieca pushed a commit that referenced this pull request Apr 8, 2021
#7259)

The platform name for MSN4600C in sfputil pliugin is not complete: "x86_64-mlnx_msn4600c" -> "x86_64-mlnx_msn4600c-r0"

Signed-off-by: Kebo Liu <kebol@nvidia.com>
@keboliu keboliu deleted the fix-4600c-typo branch April 9, 2021 02:35
raphaelt-nvidia pushed a commit to raphaelt-nvidia/sonic-buildimage that referenced this pull request May 23, 2021
sonic-net#7259)

The platform name for MSN4600C in sfputil pliugin is not complete: "x86_64-mlnx_msn4600c" -> "x86_64-mlnx_msn4600c-r0"

Signed-off-by: Kebo Liu <kebol@nvidia.com>
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
sonic-net#7259)

The platform name for MSN4600C in sfputil pliugin is not complete: "x86_64-mlnx_msn4600c" -> "x86_64-mlnx_msn4600c-r0"

Signed-off-by: Kebo Liu <kebol@nvidia.com>
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.

'sfputil reset' raises exception
4 participants