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:17483] [Nokia-IXR7250E] Modify the platform_reboot on the IXR7250E for PMON API reboot and disable all SFPs #17727

Merged
merged 1 commit into from
Jan 10, 2024

Conversation

mssonicbld
Copy link
Collaborator

Why I did it

When Supervisor card is rebooted by using PMON API, it takes about 90 seconds to trigger the shutdown in down path. At this time linecards have been up. This delays linecards database initialization which is trying to PING/PONG the database-chassis. To address this issue, we modified the NDK to use the system call with "sudo reboot" when the request is from PMON API on Supervisor case. The NDK version is 22.9.20 and greater. This new NDK requires this modifcaiton of platform_reboot to work with.

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

How I did it

Modify the platform_reboot In Supervisor not to reboot all IMMs since it has been done in the function reboot() in module.py. Also handle the reboot-cause.txt for on the Supervisor when the reboot is request from PMON API.
Modify the Nokia platform specific platform_reboot in linecard to disable all SPFs.
This PR works with NDK version 22.9.20 and above

How to verify it

  1. On supervisor card, execute the reboot() method in module.py to reboot the Supervisor card.
  2. Verify the all linecards have been rebooted successfully.
  3. Execute the show reboot-cause history command on Supervisor card after it is up.
  4. The following entries with cause "reboot from PMON" should be shown in the table:
admin@ixre-cpm-chassis7:~$ show reboot-cause history 
Name Cause Time User Comment
------------------- ---------------- ------------------------------- -------- ----------------------------------------------------------------------------------------------
2023_12_12_16_19_03 reboot from PMON Tue 12 Dec 2023 04:17:03 PM UTC PMON-API N/A
2023_12_12_15_41_30 reboot Tue 12 Dec 2023 03:39:08 PM UTC admin N/A
2023_12_12_08_42_47 reboot Tue 12 Dec 2023 08:40:37 AM UTC admin N/A
2023_12_12_05_03_57 reboot from PMON Tue 12 Dec 2023 05:01:48 AM UTC PMON-API N/A
2023_12_12_04_45_56 reboot Tue 12 Dec 2023 04:43:42 AM UTC admin N/A
2023_12_12_04_36_48 reboot from PMON Tue 12 Dec 2023 04:34:27 AM UTC PMON-API N/A
2023_12_12_04_33_25 reboot Tue 12 Dec 2023 04:31:13 AM UTC admin N/A

  1. Execute sudo reboot on a Linecard. Verify the peer ports detect the link down while the platform specific reboot is running.

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)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

…API reboot and Disable all SFPs (sonic-net#17483)

Why I did it
When Supervisor card is rebooted by using PMON API, it takes about 90 seconds to trigger the shutdown in down path. At this time linecards have been up. This delays linecards database initialization which is trying to PING/PONG the database-chassis. To address this issue, we modified the NDK to use the system call with "sudo reboot" when the request is from PMON API on Supervisor case. The NDK version is 22.9.20 and greater. This new NDK requires this modifcaiton of platform_reboot to work with.

Work item tracking
Microsoft ADO (number only): 26365734
How I did it
Modify the platform_reboot In Supervisor not to reboot all IMMs since it has been done in the function reboot() in module.py. Also handle the reboot-cause.txt for on the Supervisor when the reboot is request from PMON API.
Modify the Nokia platform specific platform_reboot in linecard to disable all SPFs.
This PR works with NDK version 22.9.20 and above

Signed-off-by: mlok <marty.lok@nokia.com>
@mssonicbld
Copy link
Collaborator Author

Original PR: #17483

@mssonicbld mssonicbld merged commit 88ee9f7 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