Add config for metrics port (#18)

fabric8-services · Nov 9, 2018 · 46a6522 · 46a6522
1 parent 746ab9a
commit 46a6522
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/config-template.yml b/config-template.yml
@@ -31,3 +31,7 @@ sentry:
 server:
   host: "localhost"
   port: 8099
+# This section defines the url used by metrics.
+Metrics:
+  host: "localhost"
+  port: 8099
diff --git a/main.go b/main.go
@@ -80,13 +80,14 @@ func main() {
 	if configuration.Metrics.Port == configuration.Server.Port {
 		http.Handle("/metrics", promhttp.Handler())
 	} else {
+
 		go func(metricAddress string) {
 			mx := http.NewServeMux()
 			mx.Handle("/metrics", promhttp.Handler())
 			if err := http.ListenAndServe(metricAddress, mx); err != nil {
 				service.LogError("startup", "err", err)
 			}
-		}(":" + strconv.Itoa(configuration.Server.Port)) // TODO FIX CONFIG
+		}(":" + strconv.Itoa(configuration.Metrics.Port)) // if no port defined in config file, pick one available.
 	}
 
 	// Start service