fix(metrics): add ending dot for graphite prefix

conf/defaults.ini

@@ -355,7 +355,7 @@ interval_seconds  = 60
 # Send internal Grafana metrics to graphite
 # Send internal Grafana metrics to graphite
 ; [metrics.graphite]
 ; [metrics.graphite]
 ; address = localhost:2003
 ; address = localhost:2003
-; prefix = service.grafana.%(instance_name)s
+; prefix = service.grafana.%(instance_name)s.
 
 
 [grafana_net]
 [grafana_net]
 url = https://grafana.net
 url = https://grafana.net

conf/sample.ini

@@ -305,7 +305,7 @@ enabled           = true
 # Send internal metrics to Graphite
 # Send internal metrics to Graphite
 ; [metrics.graphite]
 ; [metrics.graphite]
 ; address = localhost:2003
 ; address = localhost:2003
-; prefix = service.grafana.%(instance_name)s
+; prefix = service.grafana.%(instance_name)s.
 
 
 #################################### Internal Grafana Metrics ##########################
 #################################### Internal Grafana Metrics ##########################
 # Url used to to import dashboards directly from Grafana.net
 # Url used to to import dashboards directly from Grafana.net

pkg/metrics/graphite.go

@@ -33,7 +33,7 @@ func CreateGraphitePublisher() (*GraphitePublisher, error) {
 	prefix := graphiteSection.Key("prefix").Value()
 	prefix := graphiteSection.Key("prefix").Value()
 
 
 	if prefix == "" {
 	if prefix == "" {
-		prefix = "service.grafana.%(instance_name)s"
+		prefix = "service.grafana.%(instance_name)s."
 	}
 	}
 
 
 	publisher.prefix = strings.Replace(prefix, "%(instance_name)s", safeInstanceName, -1)
 	publisher.prefix = strings.Replace(prefix, "%(instance_name)s", safeInstanceName, -1)

pkg/metrics/graphite_test.go

@@ -19,8 +19,8 @@ func TestGraphitePublisher(t *testing.T) {
 		So(err, ShouldBeNil)
 		So(err, ShouldBeNil)
 
 
 		sec, err := setting.Cfg.NewSection("metrics.graphite")
 		sec, err := setting.Cfg.NewSection("metrics.graphite")
-		sec.NewKey("prefix", "service.grafana.%(instance_name)s")
-		sec.NewKey("address", "localhost:2003")
+		sec.NewKey("prefix", "service.grafana.%(instance_name)s.")
+		sec.NewKey("address", "localhost:2001")
 
 
 		So(err, ShouldBeNil)
 		So(err, ShouldBeNil)
 
 
@@ -30,7 +30,8 @@ func TestGraphitePublisher(t *testing.T) {
 		So(err, ShouldBeNil)
 		So(err, ShouldBeNil)
 		So(publisher, ShouldNotBeNil)
 		So(publisher, ShouldNotBeNil)
 
 
-		So(publisher.prefix, ShouldEqual, "service.grafana.hostname_with_dots_com")
+		So(publisher.prefix, ShouldEqual, "service.grafana.hostname_with_dots_com.")
+		So(publisher.address, ShouldEqual, "localhost:2001")
 	})
 	})
 
 
 	Convey("Test graphite publisher default values", t, func() {
 	Convey("Test graphite publisher default values", t, func() {
@@ -49,7 +50,7 @@ func TestGraphitePublisher(t *testing.T) {
 		So(err, ShouldBeNil)
 		So(err, ShouldBeNil)
 		So(publisher, ShouldNotBeNil)
 		So(publisher, ShouldNotBeNil)
 
 
-		So(publisher.prefix, ShouldEqual, "service.grafana.hostname_with_dots_com")
+		So(publisher.prefix, ShouldEqual, "service.grafana.hostname_with_dots_com.")
 		So(publisher.address, ShouldEqual, "localhost:2003")
 		So(publisher.address, ShouldEqual, "localhost:2003")
 	})
 	})
 }
 }