Bug 440486

Summary: bug report and improvement request: add more/better SMART-infos?
Product: [Plasma] plasma-disks Reporter: slartibart70
Component: generalAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: wishlist CC: nate, sitter
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Fedora RPMs   
OS: Linux   
Latest Commit: Version Fixed In: 5.23

Description slartibart70 2021-08-01 10:38:13 UTC
Hi,
the SMART status in the info-center (detailed information: Devices, then select 'smart status' just shows the text
"this device appears to be working as expected"

There is a link to the partition manager, i follow this, select a device and go to 'smart status' of the device which will produce a popup with just a minor amount of smart infos (10 items, to be exact)
Important details like power cycles are zero (?) and 'powered on' is zero as well?
(tested on two nvme drives)

This is in stark contrast to the information output by 'smartctl' for those aforementioned devices.

so far for the bugreport, now on to the improvements:

Just a question: whoever came up with the output idea saying (info-center): 
"this device appears to be working as expected"

Are we Windows-like now? We can surely do better, we have all the specific details to make this single output line waaay better. Or, if this is really meant to be this 'short', we should improve the smart details of kde partition manager.

I know, there is is gnome-disks able to display (and interpret) the smart data, why not using this? (yep, true, but when we have plasma apps, shouldn't they be similar to other tools in information detail, especially when options like 'smart details' exist?)

Or, i just stick to the commandline toolset (smartctl). This is fine for me, it also means less clickedy-click.
But, i think we should offer more choice and more data to the users of plasma tools.

Your opinion?
Comment 1 Harald Sitter 2021-09-14 11:09:25 UTC
Git commit f2b45646fd8e58775c9257e94fe0d43049d40f72 by Harald Sitter.
Committed on 14/09/2021 at 11:09.
Pushed by sitter into branch 'master'.

add magnificent advanced data

to allow users to get more detailed smartctl reports we'll now also
fetch the cli output in addition to the json output. this allows us to
show the verbatim output without much hassle to give knowledge users the
necessary info to actually figure out what exactly is wrong. also this
using a textarea so it may also be easily copied by not advanced users
and pasted on forums or other support channels to ask for help

A  +60   -0    autotests/fixtures/cli.txt
A  +2    -0    autotests/fixtures/cli.txt.license
M  +1    -0    autotests/fixtures/simulation.qrc
M  +10   -1    autotests/smartmonitortest.cpp
M  +16   -0    src/device.cpp
M  +6    -0    src/device.h
M  +23   -5    src/helper.cpp
M  +10   -0    src/kcm/package/contents/main.qml
A  +18   -0    src/kcm/package/contents/ui/ReportPage.qml     [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)]
M  +9    -1    src/simulationctl.cpp
M  +1    -1    src/smartctl.cpp
M  +7    -2    src/smartctl.h
M  +3    -1    src/smartmonitor.cpp
M  +1    -1    src/smartmonitor.h

https://invent.kde.org/plasma/plasma-disks/commit/f2b45646fd8e58775c9257e94fe0d43049d40f72