fix issue parsing metric names (#15)

CHANGELOG.md

@@ -1,3 +1,10 @@
+## v1.7.0.sfx1 [2018-06-27]
+
+### Release Notes
+
+- Fix bug in SignalFx Output Plugin that resulted in metrics with "value" field
+  to be dropped
+
 ## v1.7.0.sfx0 [2018-06-22]
 
 ### Release Notes

plugins/outputs/signalfx/parse/parse.go

@@ -21,18 +21,18 @@ func SetPluginDimension(metricName string, metricDims map[string]string) {
 }
 
 // GetMetricName combines telegraf fields and tags into a full metric name
-func GetMetricName(metric string, field string, dims map[string]string) (name string, isSFX bool) {
+func GetMetricName(metric string, field string, dims map[string]string) (string, bool) {
 	// If sf_metric is provided
-	if name, isSFX = dims["sf_metric"]; isSFX {
-		return
+	if name, isSFX := dims["sf_metric"]; isSFX {
+		return name, isSFX
 	}
 
 	// Include field when it adds to the metric name
 	if field != "value" {
-		name = metric + "." + field
+		return fmt.Sprintf("%s.%s", metric, field), false
 	}
 
-	return
+	return metric, false
 }
 
 // ExtractProperty of the metric according to the following rules

plugins/outputs/signalfx/signalfx.go

@@ -231,7 +231,7 @@ func (s *SignalFx) GetObjects(metrics []telegraf.Metric, dps chan *datapoint.Dat
 
 			// Check if the metric is explicitly excluded
 			if s.isExcluded(metricName) {
-				log.Println("D! Outputs [signalfx] excluding the following metric: ", metricName)
+				log.Println("D! Outputs [signalfx] excluding the following metric: ", metricName, metric)
 				continue
 			}