Refactor value verification to manage bools.

Strings are still dropped, bools are set to 1 for true and 0 for false
Solves: influxdata#5006

plugins/serializers/splunkmetric/splunkmetric.go

@@ -68,10 +68,19 @@ func (s *serializer) createObject(metric telegraf.Metric) (metricGroup []byte, e
 
 	for _, field := range metric.FieldList() {
 
-		if !verifyValue(field.Value) {
-			log.Printf("D! Can not parse value: %v for key: %v", field.Value, field.Key)
-			continue
-		}
+	    switch field.Value.(type) {
+            case string:
+			    log.Printf("D! Can not parse value: %v for key: %v", field.Value, field.Key)
+                continue
+            case bool:
+                if field.Value == bool(true) {
+                    // Store 1 for a "true" value
+                    field.Value = 1
+                } else {
+                    // Otherwise store 0
+                    field.Value = 0
+                }
+        }
 
 		obj := map[string]interface{}{}
 		obj["metric_name"] = metric.Name() + "." + field.Key
@@ -116,11 +125,3 @@ func (s *serializer) createObject(metric telegraf.Metric) (metricGroup []byte, e
 
 	return metricGroup, nil
 }
-
-func verifyValue(v interface{}) bool {
-	switch v.(type) {
-	case string:
-		return false
-	}
-	return true
-}

plugins/serializers/splunkmetric/splunkmetric_test.go

@@ -97,6 +97,46 @@ func TestSerializeMetricIntHec(t *testing.T) {
 	assert.Equal(t, string(expS), string(buf))
 }
 
+func TestSerializeMetricBool(t *testing.T) {
+	now := time.Unix(0, 0)
+	tags := map[string]string{
+		"container-name": "telegraf-test",
+	}
+	fields := map[string]interface{}{
+		"oomkiller": bool(true),
+	}
+	m, err := metric.New("docker", tags, fields, now)
+	assert.NoError(t, err)
+
+	s, _ := NewSerializer(false)
+	var buf []byte
+	buf, err = s.Serialize(m)
+	assert.NoError(t, err)
+
+    expS := `{"_value":1,"container-name":"telegraf-test","metric_name":"docker.oomkiller","time":0}`
+	assert.Equal(t, string(expS), string(buf))
+}
+
+func TestSerializeMetricBoolHec(t *testing.T) {
+	now := time.Unix(0, 0)
+	tags := map[string]string{
+		"container-name": "telegraf-test",
+	}
+	fields := map[string]interface{}{
+		"oomkiller": bool(false),
+	}
+	m, err := metric.New("docker", tags, fields, now)
+	assert.NoError(t, err)
+
+	s, _ := NewSerializer(true)
+	var buf []byte
+	buf, err = s.Serialize(m)
+	assert.NoError(t, err)
+
+    expS := `{"time":0,"event":"metric","fields":{"_value":0,"container-name":"telegraf-test","metric_name":"docker.oomkiller"}}`
+	assert.Equal(t, string(expS), string(buf))
+}
+
 func TestSerializeMetricString(t *testing.T) {
 	now := time.Unix(0, 0)
 	tags := map[string]string{