[202205] Fix issue: should use 'Value' column to calculate the health percentage for Virtium SSD#385
Merged
yxieca merged 3 commits intosonic-net:202205from Sep 12, 2023
Conversation
keboliu
approved these changes
Jul 11, 2023
- Description Deduce SSD vendor name from part number for Virtum - Motivation and Context Currently, ssd_generic.py deduce vendor name by smartctl command. For example, Device Model: StorFly VSFDM8XC240G-V11-T "StorFly" is the vendor name. However, for some SSD vendor, smartctl cannot get vendor name. For example: Device Model: VSFDM8XC240G-V11-T In such case, vendor name shall be deduced from part number.
…rtium SSD (sonic-net#344) * [ssd_generic] Get health status from Remaining_Life_Left field for virtium SSD * Improve UT cover * Fix UT failure
Junchao-Mellanox
force-pushed
the
202205-fix-ssd-health
branch
from
5d12eb0 to
07ccd27
Compare
Contributor
Author
Collaborator
prgeor
approved these changes
Aug 22, 2023
Collaborator
|
@yxieca suggest to merge to 202205. can you please review? |
Collaborator
|
@yxieca, it is CLI only and doesn't change results of any sonic-mgmt test |
oleksandrivantsiv
pushed a commit
to oleksandrivantsiv/sonic-platform-common
that referenced
this pull request
Oct 25, 2024
sonic-net#385) * Optics SI settings changes for platform daemon
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Backport commits:
07ccd27 Fix issue: should use 'Value' column to calculate the health percentage
6d2d7a3 [ssd_generic] Get health status from Remaining_Life_Left field for virtium SSD (#344)
feb5036 [SSD] deduce vendor name from part number (#322)
Description
Currently, we are using "Low Raw" column to calculate SSD health, but "Value"column should be used according to vendor document.
Motivation and Context
Fix issue that SSD health display wrong value
How Has This Been Tested?
Manual test
Unit test
Additional Information (Optional)