Skip to content

Commit

Permalink
fix(inputs.smartctl): Use --scan-open instead of --scan to provide co…
Browse files Browse the repository at this point in the history
…rrect device type info (#15724)
  • Loading branch information
TTTPOB authored Aug 12, 2024
1 parent 27db10e commit 706e922
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions plugins/inputs/smartctl/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ See the [CONFIGURATION.md][CONFIGURATION.md] for more details.

## Devices to include or exclude
## By default, the plugin will use all devices found in the output of
## `smartctl --scan`. Only one option is allowed at a time. If set, include
## `smartctl --scan-open`. Only one option is allowed at a time. If set, include
## sets the specific devices to scan, while exclude omits specific devices.
# devices_include = []
# devices_exclude = []
Expand Down Expand Up @@ -96,7 +96,7 @@ Defaults!SMARTCTL !logfile, !syslog, !pam_session
This plugin uses the following commands to determine devices and collect
metrics:

* `smartctl --json --scan`
* `smartctl --json --scan-open`
* `smartctl --json --all $DEVICE --device $TYPE --nocheck=$NOCHECK`

Please include the output of the above two commands for all devices that are
Expand Down
2 changes: 1 addition & 1 deletion plugins/inputs/smartctl/sample.conf
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@

## Devices to include or exclude
## By default, the plugin will use all devices found in the output of
## `smartctl --scan`. Only one option is allowed at a time. If set, include
## `smartctl --scan-open`. Only one option is allowed at a time. If set, include
## sets the specific devices to scan, while exclude omits specific devices.
# devices_include = []
# devices_exclude = []
Expand Down
2 changes: 1 addition & 1 deletion plugins/inputs/smartctl/smartctl_scan.go
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import (
)

// This is here so we can override it during testing
var scanArgs = []string{"--json", "--scan"}
var scanArgs = []string{"--json", "--scan-open"}

type scanDevice struct {
Name string
Expand Down

0 comments on commit 706e922

Please sign in to comment.