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.

Sign up for GitHub

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

Jump to bottom

no SMART info on MSA70 #997

Closed
phillxnet opened this issue Nov 3, 2015 · 5 comments
Closed

no SMART info on MSA70 #997

phillxnet opened this issue Nov 3, 2015 · 5 comments
Assignees
@phillxnet
Labels
enhancement
Milestone
Kilauea Iki

Comments

@phillxnet
Copy link
Member

Thanks to @titus_36 on the forum for reporting this.
It seems there is no WebUI SMART info displayed yet it is available on the command line.
Reason seems to be that the info is differently formatted to what the current parser expects and there is much less of it. Original reporter is hopefully to send in no-smart-msa70-report.tar.gz file resulting form commands requested on the following forum thread:- http://forum.rockstor.com/t/no-smart-support-for-msa70/641

Please update the forum post on progress with this issue.
@schakrava schakrava added this to the Kilauea Iki milestone Dec 12, 2015
@phillxnet
Copy link
Member Author

I think we have another example of the same problem in the following forum post:-
http://forum.rockstor.com/t/issues-cant-wipe-some-drives-no-smart-diagnostics-disk-serial-numbers-not-legitamate/820
Please also update this forum thread as well as the original with any progress on this issue.
Thanks to @priel71 in the referenced thread for also reporting this.
I have asked similarly that a tgz or the relevant smart output be sent in so that parsing issues can be resolved if in fact that is the cause. In this case no-smart-lsi9207-report.tar.gz .

@phillxnet
Copy link
Member Author

Not yet sure if this issue is related to smart available but not enabled via command line report but GUI reporting not available. See forum thread http://forum.rockstor.com/t/s-m-a-r-t-support-is-not-available-for-this-disk/830

@phillxnet
Copy link
Member Author

I am currently looking into this issue:

Found issues using submitted info from lsi9207 smart reporting:-
Initial take on problems:-
-a (attributes tab) gives no attributes where as normally it lists a bunch of them.
-c (capabilities tab) gives no info
--info (used to asses availability and if smart is enabled) should work but looks to have a string matching bug for the submitted lsi output. N.B. used by
-H --info (used to collect main identity tag info) matching issues similar to --info above for enabled and different formatting for keys used for health. Also need to add aditional keys as they appear quite different from what we currently look for.
-l error (used by the error log tab) completely different formatting so will need to setup a test feed of the submitted smart results.
-l selftest -l selective (used for self test logs tab) Insufficent info as no tests were reported as having been run on submitted info. Will assume this report parsing is OK.

phillxnet added a commit to phillxnet/rockstor-core that referenced this issue Jan 1, 2016
@phillxnet
add todo entries and some comments for issue rockstor#997
aea63b0 
It seems some SMART reports, in this case from an lsi controller,
differs in formatting and content, this is causing smart to appear as
not available or enabled and even if this wasn't the case the reports
differ such that current parsing is challenged to find relevant entries.
phillxnet added a commit to phillxnet/rockstor-core that referenced this issue Jan 1, 2016
@phillxnet
add test mode to smart available function rockstor#997
99ff7bd 
Add a mode to read an example smartctl output file dump
read via cat instead of the actual smartctl command. This
enables test parsing of submitted smartctl output.
phillxnet added a commit to phillxnet/rockstor-core that referenced this issue Jan 1, 2016
@phillxnet
improve sensing of smart available / enabled status rockstor#997
58207c8 
some smart reports appear to have additional spaces after the
: char so allow matches for this to enable smart status read.
phillxnet added a commit to phillxnet/rockstor-core that referenced this issue Jan 1, 2016
@phillxnet
add comments and test mode to smart info function rockstor#997
ac27d5e 
Plus some additional temporary logging.
phillxnet added a commit to phillxnet/rockstor-core that referenced this issue Jan 1, 2016
@phillxnet
improve string matching to populate Identity tab rockstor#997
3c5ebe8 
Add additional string matches to catch equivalent info from
some differently named SMART attributes.
Some controllers / hw seem to name attributes differently,
ie "Serial number" or "Serial Number", "Firmware" or "Revision"
phillxnet added a commit to phillxnet/rockstor-core that referenced this issue Jan 1, 2016
@phillxnet
code reformatting - note quite PEP8 yet though rockstor#997
76f8689
phillxnet added a commit to phillxnet/rockstor-core that referenced this issue Jan 1, 2016
@phillxnet
add test mode to smart extended_info / attributes parser rockstor#997
17e68d0 
Add more comments to explain the use of more parsers.
phillxnet added a commit to phillxnet/rockstor-core that referenced this issue Jan 1, 2016
@phillxnet
add test mode and comments to smart capabilities parser rockstor#997
39c0e70 
Removed previous logging and improved comments on other parsers
in the same file.
phillxnet added a commit to phillxnet/rockstor-core that referenced this issue Jan 1, 2016
@phillxnet
add test mode and comments to smart error_logs parser rockstor#997
d632dc2 
add temporary logging to assess test mode.
phillxnet added a commit to phillxnet/rockstor-core that referenced this issue Jan 1, 2016
@phillxnet
add todo for pointer on picking up issue rockstor#997
bc7ab23 
Note on where I left off with error_logs. Parser works
but later GUI component need tweak to at least display
the raw logs.
@phillxnet
Copy link
Member Author

I will try and pick this up again tomorrow (UK). Have fixed smart availability / enabled status reading bug on some differently formatted smart outputs such as supplied lsi9207 but need to do more work on parsing and displaying the same differently formatted smart output.

I expect this to be ready soon for pr / review stage.

phillxnet added a commit to phillxnet/rockstor-core that referenced this issue Jan 2, 2016
@phillxnet
enhance display of smart error log to be more flexible rockstor#997
f8a921a 
Some smart reports have no standard formatting, in this case
indicate that no summary is available and display the raw log
if > 4 lines, taking care to preserving the original formatting.
phillxnet added a commit to phillxnet/rockstor-core that referenced this issue Jan 2, 2016
@phillxnet
indicate when no smart capabilities found rockstor#997
2152752 
Some drives report no capabilities as this is an ATA/SATA
only feature so state non found rather than empty table.
Also reformat to spaces not tabs
phillxnet added a commit to phillxnet/rockstor-core that referenced this issue Jan 2, 2016
@phillxnet
indicate when no smart attributes found rockstor#997
352cdad 
Some drives report no attributes as this is an ATA/SATA
only feature so state non found rather than empty table.
Also reformat related code.
phillxnet added a commit to phillxnet/rockstor-core that referenced this issue Jan 2, 2016
@phillxnet
remove smart error_logs logging / todo for issue rockstor#997
f0dc3db
phillxnet added a commit to phillxnet/rockstor-core that referenced this issue Jan 2, 2016
@phillxnet
preserve formatting of smart self-test logs in UI rockstor#997
3e413b3
phillxnet added a commit to phillxnet/rockstor-core that referenced this issue Jan 2, 2016
@phillxnet
add test mode and comments to smart test_logs rockstor#997
1856f28
phillxnet added a commit to phillxnet/rockstor-core that referenced this issue Jan 2, 2016
@phillxnet
set smart TESTMODE False, add explanation, remove todos rockstor#997
cd1101e
@phillxnet phillxnet mentioned this issue Jan 2, 2016
Merged
@phillxnet
Copy link
Member Author

@schakrava Pull request ready for review.

schakrava added a commit that referenced this issue Jan 4, 2016
@schakrava
Merge pull request #1076 from phillxnet/997_no_SMART_info_on_MSA70
a122da4 
no smart info on msa70. Fixes #997
@phillxnet phillxnet mentioned this issue Jan 10, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@phillxnet phillxnet

Labels
enhancement
Projects
None yet
Milestone
Kilauea Iki
Development

No branches or pull requests
2 participants
@schakrava @phillxnet