Scrutiny (Full Access) - error during startup

[pxFIN]

I have been using device type sat in previous version 0.7.x. now error message during startup

ref.... alexbelgium/hassio-addons#1308

-----------------------------------------------------------
 Add-on: Scrutiny (Full Access)
 Scrutiny WebUI for smartd S.M.A.R.T monitoring (Full Access)
-----------------------------------------------------------
 Add-on version: v0.8.0-2
 You are running the latest version of this add-on.
 System: Home Assistant OS 12.1  (aarch64 / raspberrypi4-64)
 Home Assistant Core: 2024.3.1
 Home Assistant Supervisor: 2024.03.0
-----------------------------------------------------------
 Please, share the above information when looking for help
 or support in, e.g., GitHub, forums
-----------------------------------------------------------

time="2024-03-17T16:13:16+02:00" level=info msg="Verifying required tools" type=metrics
time="2024-03-17T16:13:16+02:00" level=info msg="Executing command: /usr/sbin/smartctl --scan --json --dev sat" type=metrics
time="2024-03-17T16:13:16+02:00" level=info msg="Executing command: /usr/sbin/smartctl --info --json --dev sat /dev/sda" type=metrics
time="2024-03-17T16:13:16+02:00" level=info msg="Generating WWN" type=metrics
time="2024-03-17T16:13:16+02:00" level=info msg="Sending detected devices to API, for filtering & validation" type=metrics
time="2024-03-17T16:13:16+02:00" level=info msg="127.0.0.1 - db21ed7f-scrutiny-fa [17/Mar/2024:16:13:16 +0200] \"POST /api/devices/register\" 200 637 \"\" \"Go-http-client/1.1\" (3ms)" clientIP=127.0.0.1 hostname=db21ed7f-scrutiny-fa latency=3 method=POST path=/api/devices/register referer= respLength=637 statusCode=200 type=web userAgent=Go-http-client/1.1
time="2024-03-17T16:13:16+02:00" level=info msg="Collecting smartctl results for sda\n" type=metrics
time="2024-03-17T16:13:16+02:00" level=info msg="Executing command: /usr/sbin/smartctl --xall --json --dev sat --device sat /dev/sda" type=metrics
time="2024-03-17T16:13:16+02:00" level=error msg="smartctl returned an error code (1) while processing sda\n" type=metrics
time="2024-03-17T16:13:16+02:00" level=error msg="smartctl could not parse commandline" type=metrics
time="2024-03-17T16:13:16+02:00" level=info msg="Publishing smartctl results for 0x5002538e30823ba8\n" type=metrics
time="2024-03-17T16:13:16+02:00" level=error msg="An error occurred while saving smartctl metrics unprocessable entity: failure writing points to database: partial write: points beyond retention policy dropped=1" type=web

[pxFIN]

Looks like multiple dev switches used for command line smartctl ?

root@db21ed7f-scrutiny-fa:/opt/scrutiny# smartctl --xall --json --dev sat --device sat /dev/sda

{
  "json_format_version": [
    1,
    0
  ],
  "smartctl": {
    "version": [
      7,
      3
    ],
    "svn_revision": "5338",
    "platform_info": "aarch64-linux-6.1.73-haos-raspi",
    "build_info": "(local build)",
    "argv": [
      "smartctl",
      "--xall",
      "--json",
      "--dev",
      "sat",
      "--device",
      "sat",
      "/dev/sda"
    ],
    "messages": [
      {
        "string": "ERROR: multiple -d TYPE options are only allowed with --scan",
        "severity": "error"
      }
    ],
    "exit_status": 1
  }
}

[AnalogJ]

Can you share your collector config file?

[pxFIN]

As a addon in HA its very simple

Updates: Hourly
Mode: Collector+WebUI
SMARTCTL_COMMAND_DEVICE_TYPE: sat

[pxFIN]

Just wondering if row 128 is meaninful

https://github.com/AnalogJ/scrutiny/blob/master/collector/pkg/collector/metrics.go

[pxFIN]

If you need specific logs.. be specific what you need

[pxFIN]

In my opinion there should be one device to call on the command line.. non standards.. scsi, ata, sat are standards

[AnalogJ]

I think this is related to the HA addon overriding the smartctl command.

https://github.com/alexbelgium/hassio-addons/blob/22ed7169a18844f6b85f8dc9210d48d05fca7bec/scrutiny/rootfs/etc/cont-init.d/01-configuration.sh#L71-L76

[pxFIN]

Is it then back the alex

[pxFIN]

Up there i was referring originally from alex and he was sure his code is good

[AnalogJ]

I responded to the original issue in alexbelgium/hassio-addons

Once he confirms the issue, can you close this? Thanks!

[pxFIN]

I responded to the original issue in alexbelgium/hassio-addons

Once he confirms the issue, can you close this? Thanks!

Of course i dont want to see open issues more than you ;-)

[pxFIN]

Thank you AnalogJ for your quick response for the issue

[pxFIN]

I've tested new version and i can see smartctl cmd although different behaviour is missing device type

Just wondering.. original 'problem' was scanning went OK but not the actual smartctl command getting the data. Are both scanning and getting data from device under same idea what you mentioned.. referring to the smartctl commands/switches --device is added conditionally .. i hope i make any sense :)

alexbelgium/hassio-addons#1308 (comment)