Ignore stdout & stderr logs and log warning (#973)

Author: aphralG

api/grpc/mpi/v1/command.pb.go

@@ -129,6 +129,10 @@ func (ncp *NginxConfigParser) createNginxConfigContext(
 					if !ncp.ignoreLog(directive.Args[0]) {
 						errorLog := ncp.errorLog(directive.Args[0], ncp.errorLogDirectiveLevel(directive))
 						nginxConfigContext.ErrorLogs = append(nginxConfigContext.ErrorLogs, errorLog)
+					} else {
+						slog.WarnContext(ctx, fmt.Sprintf("Currently error log outputs to %s. Log monitoring "+
+							"is disabled while applying a config; "+"log errors to file to enable error monitoring",
+							directive.Args[0]), "error_log", directive.Args[0])
 					}
 				case "root":
 					rootFiles := ncp.rootFiles(ctx, directive.Args[0])
@@ -185,7 +189,7 @@ func (ncp *NginxConfigParser) createNginxConfigContext(
 
 func (ncp *NginxConfigParser) ignoreLog(logPath string) bool {
 	logLower := strings.ToLower(logPath)
-	ignoreLogs := []string{"off", "/dev/stderr", "/dev/stdout", "/dev/null"}
+	ignoreLogs := []string{"off", "/dev/stderr", "/dev/stdout", "/dev/null", "stderr", "stdout"}
 
 	if strings.HasPrefix(logLower, "syslog:") || slices.Contains(ignoreLogs, logLower) {
 		return true

api/grpc/mpi/v1/common.pb.go

@@ -927,6 +927,20 @@ func TestNginxConfigParser_ignoreLog(t *testing.T) {
 			expected:    false,
 			expectedLog: "Log being read is outside of allowed directories",
 		},
+		{
+			name:        "Test 10: log stderr",
+			logPath:     "stderr",
+			excludeLogs: []string{},
+			expected:    true,
+			expectedLog: "",
+		},
+		{
+			name:        "Test 11: log stdout",
+			logPath:     "stdout",
+			excludeLogs: []string{},
+			expected:    true,
+			expectedLog: "",
+		},
 	}
 
 	for _, test := range tests {