metrics service name; metrics agent shutdown (#74, #76)

controller/config.go

@@ -7,13 +7,13 @@ import (
 )
 
 type Config struct {
-	Endpoint      *EndpointConfig
+	Endpoint     *EndpointConfig
-	Proxy         *ProxyConfig
+	Proxy        *ProxyConfig
-	Email         *EmailConfig
+	Email        *EmailConfig
-	Registration  *RegistrationConfig
+	Registration *RegistrationConfig
-	Store         *store.Config
+	Store        *store.Config
-	Ziti          *ZitiConfig
+	Ziti         *ZitiConfig
-	MetricsConfig *MetricsConfig
+	Metrics      *MetricsConfig
 }
 
 type EndpointConfig struct {

controller/controller.go

@@ -44,8 +44,14 @@ func Run(cfg *Config) error {
 		return errors.Wrap(err, "error opening store")
 	}
 
-	mtr = newMetricsAgent(cfg.MetricsConfig)
+	if cfg.Metrics != nil {
-	go mtr.run()
+		mtr = newMetricsAgent(cfg.Metrics)
+		go mtr.run()
+		defer func() {
+			mtr.stop()
+			mtr.join()
+		}()
+	}
 
 	server := rest_server_zrok.NewServer(api)
 	defer func() { _ = server.Shutdown() }()

controller/metrics.go

@@ -2,11 +2,11 @@ package controller
 
 import (
 	"github.com/sirupsen/logrus"
-	"time"
 )
 
 type MetricsConfig struct {
-	Influx *InfluxConfig
+	ServiceName string
+	Influx      *InfluxConfig
 }
 
 type InfluxConfig struct {
@@ -17,18 +17,31 @@ type InfluxConfig struct {
 }
 
 type metricsAgent struct {
-	cfg *MetricsConfig
+	cfg      *MetricsConfig
+	shutdown chan struct{}
+	joined   chan struct{}
 }
 
 func newMetricsAgent(cfg *MetricsConfig) *metricsAgent {
-	return &metricsAgent{cfg: cfg}
+	return &metricsAgent{
+		cfg:      cfg,
+		shutdown: make(chan struct{}),
+		joined:   make(chan struct{}),
+	}
 }
 
 func (mtr *metricsAgent) run() {
 	logrus.Info("starting")
 	defer logrus.Info("exiting")
+	defer close(mtr.joined)
 
-	for {
+	<-mtr.shutdown
-		time.Sleep(24 * time.Hour)
+}
-	}
+
+func (mtr *metricsAgent) stop() {
+	close(mtr.shutdown)
+}
+
+func (mtr *metricsAgent) join() {
+	<-mtr.joined
 }