23.7-RC3 SMART status widget showing extra info that does not make sense

[pmhausen]

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

• I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
• I have searched the existing issues, open and closed, and I'm convinced that mine is new.
• The title contains the plugin to which this issue belongs

Describe the bug
Since upgrading to 23.7 and still present in RC3 the SMART status widget looks like shown in the attachment.

Tip: to validate your setup was working with the previous version, use opnsense-revert (https://docs.opnsense.org/manual/opnsense_tools.html#opnsense-revert)

To Reproduce
Steps to reproduce the behavior:

1. Update to 23.7
2. Install os-smart plugin
3. Place SMART status widget on dashboard

Expected behavior
The sesX devices should not be there.

Screenshots

Kind regards,
Patrick

[fichtner]

Perhaps this broke it as theorised in the commit... f37375f4b8

I thought I annotated this but did not. Since it was a PR submission I tried to pick it up.

[pmhausen]

I see. What was wrong with the old method?

On my hardware:

root@opnsense:~ # sysctl -n kern.disks
ada1 ada0
root@opnsense:~ # smartctl --scan
/dev/ses0 -d atacam # /dev/ses0, ATA device
/dev/ada0 -d atacam # /dev/ada0, ATA device
/dev/ada1 -d atacam # /dev/ada1, ATA device
/dev/ses1 -d atacam # /dev/ses1, ATA device

[fichtner]

I‘m wondering the same, but it was a community PR so to avoid complains we can try and worst case revert.

[OPNsense-bot]

This issue has been automatically timed-out (after 180 days of inactivity).

For more information about the policies for this repository,
please read https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md for further details.

If someone wants to step up and work on this issue,
just let us know, so we can reopen the issue and assign an owner to it.