Ignore NGINX cache manager process when looking for NGINX master process (#1058)

Author: aphralG

internal/watcher/file/file_watcher_service.go

@@ -180,7 +180,6 @@ func (fws *FileWatcherService) isWatching(name string) bool {
 func (fws *FileWatcherService) handleEvent(ctx context.Context, event fsnotify.Event) {
 	if fws.enabled.Load() {
 		if fws.isEventSkippable(event) {
-			slog.DebugContext(ctx, "Skipping FSNotify event", "event", event)
 			return
 		}
 
@@ -238,7 +237,6 @@ func isExcludedFile(path string, excludeFiles []string) bool {
 			slog.Error("Invalid path for excluding file", "file_path", pattern)
 			continue
 		} else if ok {
-			slog.Debug("Excluding file from watcher as specified in config", "file_path", path)
 			return true
 		}
 	}

internal/watcher/instance/nginx_process_parser.go

@@ -103,16 +103,18 @@ func (npp *NginxProcessParser) Parse(ctx context.Context, processes []*nginxproc
 			continue
 		}
 
-		// proc is a master process
-		nginxInfo, err := npp.getInfo(ctx, proc)
-		if err != nil {
-			slog.DebugContext(ctx, "Unable to get NGINX info", "pid", proc.PID, "error", err)
+		// check if proc is a master process, process is not a worker but could be cache manager etc
+		if proc.IsMaster() {
+			nginxInfo, err := npp.getInfo(ctx, proc)
+			if err != nil {
+				slog.DebugContext(ctx, "Unable to get NGINX info", "pid", proc.PID, "error", err)
 
-			continue
-		}
+				continue
+			}
 
-		instance := convertInfoToInstance(*nginxInfo)
-		instanceMap[instance.GetInstanceMeta().GetInstanceId()] = instance
+			instance := convertInfoToInstance(*nginxInfo)
+			instanceMap[instance.GetInstanceMeta().GetInstanceId()] = instance
+		}
 	}
 
 	for _, instance := range instanceMap {

internal/watcher/instance/nginx_process_parser_test.go

@@ -352,6 +352,19 @@ func TestNginxProcessParser_Parse_Processes(t *testing.T) {
 			},
 			expected: instancesTest4,
 		},
+		{
+			name: "Test 5: 1 cache process",
+			processes: []*nginxprocess.Process{
+				{
+					PID:  1234,
+					PPID: 1,
+					Name: "nginx",
+					Cmd:  "nginx: cache manager process",
+					Exe:  exePath,
+				},
+			},
+			expected: make(map[string]*mpi.Instance),
+		},
 	}
 
 	for _, test := range tests {