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.

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

A refactor of the script. #28

Open
wants to merge 169 commits into
base: master
Choose a base branch
from
Open

Conversation

dak180
Copy link

@dak180 dak180 commented May 16, 2021

The goal is to make it easier to read, more maintainable, and to solve several outstanding issues.

To do (help welcome):

  • new screenshots.

Fixes #27.
Fixes #25.
Fixes #21.
Fixes #7.
Fixes #3.
Fixes #33.

fohlsso2 and others added 30 commits June 20, 2018 21:37
added more picture links
added more picture links
fixed the parser for normal scrub
#   - Added Frag%, Size, Allocated, Free for ZPool status report summary.
#   - Added Disk Size, RPM, Model to the Smart Report
#   - Added if statment so that if "Model Family" is not present script will use "Device Model" 
#       for brand in the SMART Satus report details.
#   - Added Glabel Status Report (RAW)
#   - Removed Power-On time labels and added ":" as a separator.
#   - Added Power-On format to the Power-On time Header.
#   - Changed Backup deafult to false.
Ids are standardized (mostly) but names are not.

Fixes #3.
* aserrallerios/master:
  Rebase v1.6 to main repo
  Fixes for TrueNAS 12.0-RELEASE Adapted script because output of zpool status changed
  added line breaks to disk info table body
Only use smart 321 when both 231 and 233 are present.
HTML boundary fix & proper message ids
Better support for NVMe and SSD
Support for new smartmon-tools
This also uses a function and is much easier to read.
Ordered by apearence.
Use current names.
Also note drives that do not support test monitoring through smartctl.

Refs #20.
512 because apparently the NVMe spec uses magic numbers.
* topic/scale:
  Add a custom message for a missing `bc`.
  Fix white space.
  -q does not work on debian.
  10^ is more portable than 1e.
  Get a more human readable status dump.
  md5sum and sha256sum are on both scale and core.
  nvmecontrol is bsd only.
  Better drive list for scale.
  Having a date field that takes almost anything is pain.
  debain date does not like _
  Account for different `date` invocations.
  Another command to test.
  First pass at linux changes; untested.
  glabel is not used on scale.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
10 participants