logging dogstatsd reporters' errors

topfreegames · Nov 7, 2017 · 5641259 · 5641259
1 parent 7b97f0f
commit 5641259
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/helm/charts/maestro/Chart.yaml b/helm/charts/maestro/Chart.yaml
@@ -1,7 +1,7 @@
 name: maestro
 home: https://github.com/topfreegames/maestro
 description: Maestro api and worker
-version: 4.4.5
+version: 4.4.6
 maintainers:
   - name: TFGCo
     email: backend@tfgco.com
diff --git a/metadata/version.go b/metadata/version.go
@@ -8,7 +8,7 @@
 package metadata
 
 //Version of Maestro
-var Version = "4.4.5"
+var Version = "4.4.6"
 
 //KubeVersion is the desired Kubernetes version
 var KubeVersion = "v1.7.5"
diff --git a/reporters/dogstatsd.go b/reporters/dogstatsd.go
@@ -20,6 +20,7 @@ import (
 // DogStatsD reports metrics to a dogstatsd.Client
 type DogStatsD struct {
 	client dogstatsd.Client
+	logger *logrus.Logger
 	region string
 }
 
@@ -34,7 +35,13 @@ func (d *DogStatsD) Report(event string, opts map[string]string) error {
 	handler := handlerI.(func(dogstatsd.Client, string, map[string]string) error)
 
 	opts[constants.TagRegion] = d.region
-	return handler(d.client, event, opts)
+
+	err := handler(d.client, event, opts)
+	if err != nil {
+		d.logger.Error(err)
+	}
+
+	return err
 }
 
 // MakeDogStatsD adds a DogStatsD struct to the Reporters' singleton