add ability to set up log output

install/helm/open-match/templates/om-configmap-default.yaml

@@ -36,6 +36,7 @@ data:
       {{- else }}
       format: text
       {{- end }}
+      output: {{ .Values.global.logging.output }}
       rpc: {{ .Values.global.logging.rpc.enabled }}
     # Open Match applies the exponential backoff strategy for its retryable gRPC calls.
     # The settings below are the default backoff configuration used in Open Match.

install/helm/open-match/values.yaml

@@ -272,6 +272,7 @@ global:
       mountPath: /app/secrets/tls/rootca
 
   logging:
+    output:
     rpc:
       enabled: false
 

internal/logging/logging.go

@@ -16,6 +16,8 @@
 package logging
 
 import (
+	"log"
+	"os"
 	"strings"
 
 	stackdriver "github.com/TV4/logrus-stackdriver-formatter"
@@ -27,6 +29,7 @@ import (
 //   - log line format (text[default] or json)
 //   - min log level to include (debug, info [default], warn, error, fatal, panic)
 func ConfigureLogging(cfg config.View) {
+	log.SetOutput(toOutput(cfg.GetString("logging.output")))
 	logrus.SetFormatter(newFormatter(cfg.GetString("logging.format")))
 	level := toLevel(cfg.GetString("logging.level"))
 	logrus.SetLevel(level)
@@ -84,3 +87,14 @@ func isDebugLevel(level logrus.Level) bool {
 	}
 	return false
 }
+
+func toOutput(output string) *os.File {
+	switch output {
+	case "stdout":
+		return os.Stdout
+	case "stderr":
+		return os.Stderr
+	default:
+		return os.Stderr
+	}
+}

internal/logging/logging_test.go

@@ -16,6 +16,7 @@ package logging
 
 import (
 	"fmt"
+	"os"
 	"reflect"
 	"testing"
 
@@ -93,3 +94,22 @@ func TestToLevel(t *testing.T) {
 		})
 	}
 }
+
+func TestToOutput(t *testing.T) {
+	testCases := []struct {
+		in       string
+		expected *os.File
+	}{
+		{"stdout", os.Stdout},
+		{"stderr", os.Stderr},
+		{"", os.Stderr},
+	}
+	for _, tc := range testCases {
+		tc := tc
+		t.Run(fmt.Sprintf("toOutput(%s) => %s", tc.in, tc.expected.Name()), func(t *testing.T) {
+			require := require.New(t)
+			actual := toOutput(tc.in)
+			require.Equal(tc.expected, actual)
+		})
+	}
+}