Merge pull request #22 from keptn-contrib/patch/improve-error-msg

Add NaN check for SLI value
keptn-contrib · Mar 5, 2020 · b526792 · b526792
2 parents 1d6a75f + 854b490
commit b526792
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/main.go b/main.go
@@ -5,6 +5,7 @@ import (
 	"errors"
 	"fmt"
 	"log"
+	"math"
 	"net/url"
 	"os"
 	"strings"
@@ -138,6 +139,13 @@ func retrieveMetrics(event cloudevents.Event) error {
 				Success: false,
 				Message: err.Error(),
 			})
+		} else if math.IsNaN(sliValue) {
+			sliResults = append(sliResults, &keptnevents.SLIResult{
+				Metric:  indicator,
+				Value:   0,
+				Success: false,
+				Message: "SLI value is NaN",
+			})
 		} else {
 			sliResults = append(sliResults, &keptnevents.SLIResult{
 				Metric:  indicator,