1 files changed, 68 insertions, 0 deletions
diff --git a/src/go/collectors/go.d.plugin/modules/smartctl/scan.go b/src/go/collectors/go.d.plugin/modules/smartctl/scan.go
new file mode 100644
index 0000000000..cafdc8b60d
--- /dev/null
+++ b/src/go/collectors/go.d.plugin/modules/smartctl/scan.go
@@ -0,0 +1,68 @@
+// SPDX-License-Identifier: GPL-3.0-or-later
+
+package smartctl
+
+import (
+	"errors"
+	"fmt"
+	"strings"
+)
+
+type scanDevice struct {
+	name     string
+	infoName string
+	typ      string
+}
+
+func (s *scanDevice) key() string {
+	return fmt.Sprintf("%s|%s", s.name, s.typ)
+}
+
+func (s *scanDevice) shortName() string {
+	return strings.TrimPrefix(s.name, "/dev/")
+}
+
+func (s *Smartctl) scanDevices() (map[string]*scanDevice, error) {
+	resp, err := s.exec.scan()
+	if err != nil {
+		return nil, err
+	}
+
+	devices := make(map[string]*scanDevice)
+
+	for _, d := range resp.Get("devices").Array() {
+		dev := &scanDevice{
+			name:     d.Get("name").String(),
+			infoName: d.Get("info_name").String(),
+			typ:      d.Get("type").String(),
+		}
+
+		if dev.name == "" || dev.typ == "" {
+			continue
+		}
+
+		if dev.typ == "scsi" {
+			// `smartctl --scan` attempts to guess the device type based on the path, but this can be unreliable.
+			// Accurate device type information is crucial because we use the `--device` option to gather data.
+			// Using the wrong type can lead to issues.
+			// For example, using 'scsi' for 'sat' devices prevents `smartctl` from issuing the necessary ATA commands.
+			resp, _ := s.exec.deviceInfo(dev.name, dev.typ, s.NoCheckPowerMode)
+			if resp != nil && isExitStatusHasBit(resp, 2) {
+				newType := "sat"
+				s.Debugf("changing device '%s' type '%s' -> '%s'", dev.name, dev.typ, newType)
+				dev.typ = newType
+			}
+			s.Debugf("smartctl scan found device '%s' type '%s' info_name '%s'", dev.name, dev.typ, dev.infoName)
+		}
+
+		devices[dev.key()] = dev
+	}
+
+	if len(devices) == 0 {
+		return nil, errors.New("no devices found on scan")
+	}
+
+	s.Infof("smartctl scan found %d devices", len(devices))
+
+	return devices, nil
+}