Ignore error when utmp is missing (#5742)

influxdata · Jun 25, 2019 · a5c94db · a5c94db
1 parent bd9ddd8
commit a5c94db
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/plugins/inputs/system/README.md b/plugins/inputs/system/README.md
@@ -13,7 +13,7 @@ and number of users logged in. It is similar to the unix `uptime` command.
 #### Permissions:
 
 The `n_users` field requires read access to `/var/run/utmp`, and may require
-the `telegraf` user to be added to the `utmp` group on some systems.
+the `telegraf` user to be added to the `utmp` group on some systems. If this file does not exist `n_users` will be skipped.
 
 ### Metrics:
 

diff --git a/plugins/inputs/system/system.go b/plugins/inputs/system/system.go
@@ -4,6 +4,7 @@ import (
 	"bufio"
 	"bytes"
 	"fmt"
+	"log"
 	"os"
 	"runtime"
 	"strings"
@@ -44,8 +45,10 @@ func (_ *SystemStats) Gather(acc telegraf.Accumulator) error {
 	users, err := host.Users()
 	if err == nil {
 		fields["n_users"] = len(users)
-	} else if !os.IsPermission(err) {
-		return err
+	} else if os.IsNotExist(err) {
+		log.Printf("D! [inputs.system] Error reading users: %v", err)
+	} else if os.IsPermission(err) {
+		log.Printf("D! [inputs.system] %v", err)
 	}
 
 	now := time.Now()