Skip to content

[201911] DellEMC S6100 SSD Monitor additional changes#7291

Merged
sujinmkang merged 2 commits intosonic-net:201911from
santhosh-kt:201911_ssd_plugin
May 4, 2021
Merged

[201911] DellEMC S6100 SSD Monitor additional changes#7291
sujinmkang merged 2 commits intosonic-net:201911from
santhosh-kt:201911_ssd_plugin

Conversation

@santhosh-kt
Copy link
Copy Markdown
Contributor

@santhosh-kt santhosh-kt commented Apr 12, 2021 •
edited
Loading

Why I did it

  • Added soft-reboot plugin support.
  • Added SSD version s16425cq check
  • Added error message to display in console/SSH in case reboot is called in faulty/non-upgraded devices.

How I did it

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

    modified:   ../../debian/platform-modules-s6100.install
    modified:   platform_reboot_pre_check
    modified:   s6100_ssd_mon.sh
    modified:   s6100_ssd_upgrade_status.sh
    new file:   soft-reboot_plugin

How to verify it

Attached logs for reference.
test-3IE3-2018-2019.txt
test-3IE-2018-2019.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)

@jleveque jleveque requested a review from sujinmkang April 14, 2021 19:13
@santhosh-kt
Copy link
Copy Markdown
Contributor Author

santhosh-kt commented Apr 22, 2021

@sujinmkang : Please dont merge the changes on sonic-buildimage PRs. One more change is on the way.

abdosi
abdosi reviewed Apr 23, 2021
View reviewed changes
Copy link
Copy Markdown
Contributor

@abdosi abdosi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

adding in loop for awareness

@santhosh-kt santhosh-kt requested a review from sujinmkang April 24, 2021 13:39
@sujinmkang
Copy link
Copy Markdown
Collaborator

sujinmkang commented May 3, 2021

@santhosh-kt this branch doesn't have the equivalent changes that master and 201811 branch have. Can you please update this branch too?

@sujinmkang sujinmkang merged commit 6204a1d into sonic-net:201911 May 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@abdosi abdosi abdosi left review comments

@sujinmkang sujinmkang Awaiting requested review from sujinmkang

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@santhosh-kt @sujinmkang @abdosi