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

[action] [PR:17378] [Nokia][sonic-platform] Update Nokia sonic-platform submodule and device data for thermal logging, etc. #17726

Merged
merged 1 commit into from
Jan 10, 2024

Conversation

mssonicbld
Copy link
Collaborator

These changes, in conjunction with NDK version >= 22.9.17 address the thermal logging issues discussed at https://github.com/Nokia-ION/ndk/issues/27. While the changes contained at this PR do not require coupling to NDK version >= 22.9.17, thermal logging enhancements will not be available without updated NDK >= 22.9.17. Thus, coupling with NDK >=22.9.17 is preferred and recommended.

Why I did it

To address thermal logging deficiencies.

Work item tracking
  • Microsoft ADO (number only): 26365734

How I did it

The following changes are included:

Threshold configuration values are provided in the associated device data .json files. There is also a change included to better handle the condition where an SFP module read fails.

Modify the module.py reboot to support reboot linecard from Supervisor

  • Modify reboot to call _reboot_imm for single IMM card reboot
  • Add log to the ndk_cmd to log the operation of "reboot-linecard" and "shutdown/satrtup the sfm"

Add new nokia_cmd set command and modify show ndk-status output

  • Add a new function reboot_imm() to nokia_common.py to support reboot a single IMM slot from CPM
  • Added new command: nokia_cmd set reboot-linecard [forece] for CPM
  • Append a new column "RebootStatus" at the end of output of "nokia_cmd show ndk-status"
  • Provide ability for IMM to disable all transceiver module TX at reboot time
  • Remove defunct xcvr-resync service

How to verify it

Verify that applicable thermal logging is occurring on both LC and Supervisor card.

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

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

Tested branch (Please provide the tested image version)

  • 202205

Description for the changelog

[Nokia][sonic-platform] Update Nokia sonic-platform submodule and device data for enhanced thermal logging : please couple with NDK >= 22.9.17

…ice data (sonic-net#17378)

These changes, in conjunction with NDK version >= 22.9.17 address the thermal logging issues discussed at Nokia-ION/ndk#27. While the changes contained at this PR do not require coupling to NDK version >= 22.9.17, thermal logging enhancements will not be available without updated NDK >= 22.9.17. Thus, coupling with NDK >=22.9.17 is preferred and recommended.

Why I did it
To address thermal logging deficiencies.

Work item tracking
Microsoft ADO (number only): 26365734
How I did it
The following changes are included:

Threshold configuration values are provided in the associated device data .json files. There is also a change included to better handle the condition where an SFP module read fails.

Modify the module.py reboot to support reboot linecard from Supervisor

 - Modify reboot to call _reboot_imm for single IMM card reboot
 - Add log to the ndk_cmd to log the operation of "reboot-linecard" and "shutdown/satrtup the sfm"
Add new nokia_cmd set command and modify show ndk-status output

 - Add a new function reboot_imm() to nokia_common.py to support reboot a single IMM slot from CPM
 - Added new command: nokia_cmd set reboot-linecard <slot> [forece] for CPM
 - Append a new column "RebootStatus" at the end of output of "nokia_cmd show ndk-status"
 - Provide ability for IMM to disable all transceiver module TX at reboot time
 - Remove defunct xcvr-resync service
@mssonicbld
Copy link
Collaborator Author

Original PR: #17378

@mssonicbld mssonicbld merged commit bcdbaf1 into sonic-net:202311 Jan 10, 2024
18 checks passed
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.

2 participants