Skip to content

[Mellanox] Remove SFP sensors from sku-sensors-data #11192

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

Conversation

slutati1536
Copy link
Contributor

@slutati1536 slutati1536 commented Jan 4, 2024

Description of PR

Align test to changes from sonic-net/sonic-buildimage#17631
The cable thermal sensors will be deprecated from the kernel driver. When cable host management is enabled, NOS will fetch the cable temperature from cable EEPROM, kernel driver will not provide the sysfs anymore.

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911
  • 202012
  • 202205
  • 202305
  • 202311

Approach

What is the motivation for this PR?

Align test to changes from sonic-net/sonic-buildimage#17631

How did you do it?

I made the changes and tested them with the build from PR

How did you verify/test it?

run sensors test

@liat-grozovik liat-grozovik changed the title Remove SFP sensors from sku-sensors-data [Mellanox] Remove SFP sensors from sku-sensors-data Feb 12, 2024
@liat-grozovik liat-grozovik merged commit 7842a0d into sonic-net:master Feb 12, 2024
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Mar 4, 2024
Align test to changes from sonic-net/sonic-buildimage#17631
The cable thermal sensors will be deprecated from the kernel driver. When cable host management is enabled, NOS will fetch the cable temperature from cable EEPROM, kernel driver will not provide the sysfs anymore.
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202311: #11857

mssonicbld pushed a commit that referenced this pull request Mar 4, 2024
Align test to changes from sonic-net/sonic-buildimage#17631
The cable thermal sensors will be deprecated from the kernel driver. When cable host management is enabled, NOS will fetch the cable temperature from cable EEPROM, kernel driver will not provide the sysfs anymore.
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