Change Virtium SSD which doesn't support SmartCMD, to use only smartctl #522
Conversation
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azpw run Azure.sonic-platform-common |
|
/AzurePipelines run Azure.sonic-platform-common |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@wangxin A kind reminder, can you please handle the failure which is caused by moving from Bullseye to Bookworm? |
|
/azpw run Azure.sonic-platform-common |
|
/AzurePipelines run Azure.sonic-platform-common |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@prgeor can you please take a look? |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
noaOrMlnx
changed the title
noaOrMlnx
force-pushed
the
special_virtium_ssd
branch
from
3604c4d to
4d16199
Compare
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
noaOrMlnx
force-pushed
the
special_virtium_ssd
branch
from
4d16199 to
46f1905
Compare
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@noaOrMlnx could you also fix output before/after change in the PR description to have uniform output format? One has |
@assrinivasan Done |
|
@bingwang-ms can you please help to check that? |
moshemos
added
Issue for 202405
Request for 202405 Branch
and removed
Issue for 202405
labels
|
@bingwang-ms @prgeor kind reminder. can you please help to merge? |
|
Cherry-pick PR to 202411: #531 |
mssonicbld
added
Included in 202411 Branch
and removed
Created PR to 202411 Branch
labels
Description
Add support for Virtium SSD which can't use SmartCMD - to use only smartctl (using generic ssd info parser).
Motivation and Context
There are Virtium SSDs which can't receive SSD info from 'SmartCMD', and must use 'smartctl' command.
For those SSDs, after calling the generic ssd parser, the function is finished and the specific vendor parser won't be called.
How Has This Been Tested?
I tested it manually with a special Virtium SSD which doesn't support SmartCMD
Additional Information (Optional)
output after the change:
root@sonic:/home/admin# show platform ssdhealth
Disk Type : NVME
Device Model : Virtium VTPM24CEXI080-BM110006
Health : 100.0%
Temperature : 52.0C
output before the change:
root@sonic:/home/admin# show platform ssdhealth
[Error] Cannot read SMART information on device /dev/nvme0n1
Disk Type : NVME
Device Model : Virtium VTPM24CEXI080-BM110006
Health : N/A
Temperature : 51.0C