[ntservice] improved checking of Windows services

- Added flag to check service names for exact match
- Fixed UNKNOWN alert if service does not exist
- Fixed that the checked service name is displayed in the message when the check result is OK

Author: tukaelu

check-ntservice/lib/check_ntservice.go

@@ -13,6 +13,7 @@ var opts struct {
 	ServiceName    string `long:"service-name" short:"s" description:"service name"`
 	ExcludeService string `long:"exclude-service" short:"x" description:"service name to exclude from matching. This option takes precedence over --service-name"`
 	ListService    bool   `long:"list-service" short:"l" description:"list service"`
+	Exact          bool   `long:"exact" description:"more exact checking of the service. This option applies only to --service-name."`
 }
 
 // Win32Service is struct for Win32_Service.
@@ -58,19 +59,26 @@ func run(args []string) *checkers.Checker {
 		return checkers.Critical(err.Error())
 	}
 
-	checkSt := checkers.OK
-	msg := ""
+	checkSt := checkers.UNKNOWN
+	msg := fmt.Sprintf("%s: service does not exist.", opts.ServiceName)
 	for _, s := range ss {
 		if opts.ExcludeService != "" && strings.Contains(s.Name, opts.ExcludeService) {
 			continue
 		}
-		if !strings.Contains(s.Name, opts.ServiceName) {
-			continue
+		if opts.Exact {
+			if s.Name != opts.ServiceName {
+				continue
+			}
+		} else {
+			if !strings.Contains(s.Name, opts.ServiceName) {
+				continue
+			}
 		}
 		if s.State == "Running" {
-			continue
+			checkSt = checkers.OK
+		} else {
+			checkSt = checkers.CRITICAL
 		}
-		checkSt = checkers.CRITICAL
 		msg = fmt.Sprintf("%s: %s - %s", s.Name, s.Caption, s.State)
 		break
 	}

check-ntservice/lib/check_ntservice_test.go

@@ -89,7 +89,7 @@ func TestRun(t *testing.T) {
 			casename:      "check about running service",
 			cmdline:       []string{"-s", "running-service"},
 			expectStatus:  checkers.OK,
-			expectMessage: "",
+			expectMessage: "running-service-name: running-service-caption - Running",
 		},
 		{
 			casename:      "check about stopped service",
@@ -101,7 +101,25 @@ func TestRun(t *testing.T) {
 			casename:      "check about running service with exclude option",
 			cmdline:       []string{"-s", "service", "-x", "stopped"},
 			expectStatus:  checkers.OK,
-			expectMessage: "",
+			expectMessage: "running-service-name: running-service-caption - Running",
+		},
+		{
+			casename:      "check about unknown service",
+			cmdline:       []string{"-s", "unknown-service-name"},
+			expectStatus:  checkers.UNKNOWN,
+			expectMessage: "unknown-service-name: service does not exist.",
+		},
+		{
+			casename:      "check about running service with --exact option",
+			cmdline:       []string{"-s", "running-service-name", "--exact"},
+			expectStatus:  checkers.OK,
+			expectMessage: "running-service-name: running-service-caption - Running",
+		},
+		{
+			casename:      "check about unmatched service with --exact option",
+			cmdline:       []string{"-s", "service", "--exact"},
+			expectStatus:  checkers.UNKNOWN,
+			expectMessage: "service: service does not exist.",
 		},
 	}