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

DellEMC S6100 SSD Monitor Porting Changes in master #7289

Merged
merged 5 commits into from
Aug 31, 2021

Conversation

santhosh-kt
Copy link
Contributor

@santhosh-kt santhosh-kt commented Apr 12, 2021

Why I did it

  • To monitor the SSD health condition in DellEMC S6100 platform post upgrade.

  • A daemon is introduced to monitor the SSD every one hour.

  • To check for SSD status at boot time and at the time of cold-reboot.

  • All these changes are supported only for newer SSD firmware.

  • Porting changes from 201911 branch

  • Added a platform_reboot_pre_check script to prevent cold-reboot based on SSD status.
    Depends on Azure/sonic-utilities#1556
    DO NOT MERGE UNTIL ABOVE PR IS MERGED

How I did it

On branch s6100_ssd_master
Changes to be committed:
(use "git reset HEAD ..." to unstage)

    modified:   ../../debian/platform-modules-s6100.install
    new file:   iSMART_64
    new file:   platform_reboot_pre_check
    modified:   s6100_platform.sh
    new file:   s6100_ssd_mon.sh
    new file:   s6100_ssd_upgrade_status.sh
    new file:   soft-reboot_plugin
    new file:   ../systemd/s6100-ssd-monitor.service
    new file:   ../systemd/s6100-ssd-monitor.timer
    new file:   ../systemd/s6100-ssd-upgrade-status.service

How to verify it

Attached logs for reference.
test-3IE3-2018-master.txt
test-3IE-2018-master.txt

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)

@santhosh-kt santhosh-kt requested a review from lguohan as a code owner April 12, 2021 04:23
@jleveque jleveque requested a review from sujinmkang April 14, 2021 19:13
sujinmkang
sujinmkang previously approved these changes Apr 20, 2021
sujinmkang
sujinmkang previously approved these changes May 3, 2021
@sujinmkang
Copy link
Collaborator

@santhosh-kt can you please resolve the conflicts?

@santhosh-kt santhosh-kt requested a review from sujinmkang August 23, 2021 05:06
@sujinmkang sujinmkang merged commit 803aa38 into sonic-net:master Aug 31, 2021
sujinmkang pushed a commit that referenced this pull request Jun 15, 2022
Why I did it
To port DellEMC S6100 SSD upgrade status checker changes from master (based on #7289) to 201811 branch

Handle newer SSD firmware version (S210506G - 3IE devices)
Recover SSD upgrade state if in case ssd_fw_upgrade folder got deleted
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants