chore: remove unused functions (#2260)

* chore: lint fixes

checks/alertmanager.go

@@ -67,7 +67,7 @@ func (c *AlertManagerChecker) Check(ctx *context.Context, extConfig external.Che
 		Filter:  filters,
 	})
 	if err != nil {
-		results.ErrorMessage(fmt.Errorf("Error fetching from alertmanager: %v", err))
+		results.ErrorMessage(fmt.Errorf("error fetching from alertmanager: %v", err))
 		return results
 	}
 

checks/database_backup.go

@@ -1,7 +1,6 @@
 package checks
 
 import (
-	"github.com/pkg/errors"
 	"github.com/prometheus/client_golang/prometheus"
 
 	"github.com/flanksource/canary-checker/api/context"
@@ -52,13 +51,6 @@ func (c *DatabaseBackupChecker) Check(ctx *context.Context, extConfig external.C
 	case check.GCP != nil:
 		return GCPDatabaseBackupCheck(ctx, check)
 	default:
-		return FailDatabaseBackupParse(ctx, check)
+		return pkg.Invalid(check, ctx.Canary, "GCP details not provided in check")
 	}
 }
-
-func FailDatabaseBackupParse(ctx *context.Context, check v1.DatabaseBackupCheck) pkg.Results {
-	result := pkg.Fail(check, ctx.Canary)
-	var results pkg.Results
-	results = append(results, result)
-	return results.ErrorMessage(errors.New("Could not parse databaseBackup input"))
-}

checks/elasticsearch.go

@@ -56,7 +56,6 @@ func (c *ElasticsearchChecker) Check(ctx *context.Context, extConfig external.Ch
 		es.Search.WithIndex(check.Index),
 		es.Search.WithBody(body),
 	)
-
 	if err != nil {
 		return results.ErrorMessage(err)
 	}
@@ -65,10 +64,10 @@ func (c *ElasticsearchChecker) Check(ctx *context.Context, extConfig external.Ch
 		var e map[string]any
 		if err := json.NewDecoder(res.Body).Decode(&e); err != nil {
 			return results.ErrorMessage(
-				fmt.Errorf("Error parsing the response body: %s", err),
+				fmt.Errorf("error parsing the response body: %s", err),
 			)
 		} else {
-			return results.ErrorMessage(fmt.Errorf("Error from elasticsearch [%s]: %v, %v",
+			return results.ErrorMessage(fmt.Errorf("error from elasticsearch [%s]: %v, %v",
 				res.Status(),
 				e["error"].(map[string]any)["type"],
 				e["error"].(map[string]any)["reason"],
@@ -82,7 +81,7 @@ func (c *ElasticsearchChecker) Check(ctx *context.Context, extConfig external.Ch
 	var r map[string]any
 	if err := json.NewDecoder(res.Body).Decode(&r); err != nil {
 		return results.ErrorMessage(
-			fmt.Errorf("Error parsing the response body: %s", err),
+			fmt.Errorf("error parsing the response body: %s", err),
 		)
 	}
 

checks/namespace.go

@@ -49,15 +49,11 @@ func NewNamespaceChecker() *NamespaceChecker {
 // Returns check result and metrics
 func (c *NamespaceChecker) Run(ctx *context.Context) pkg.Results {
 	logger.Warnf("namespace check is deprecated. Please use the kubernetes resource check")
-	var err error
 	var results pkg.Results
 	for _, conf := range ctx.Canary.Spec.Namespace {
 		if c.k8s == nil {
 			c.k8s = ctx.Kubernetes()
 			c.ctx = ctx
-			if err != nil {
-				return pkg.SetupError(ctx.Canary, err)
-			}
 		}
 		results = append(results, c.Check(c.ctx, conf)...)
 	}

checks/utils.go

@@ -3,21 +3,11 @@ package checks
 import (
 	"fmt"
 	"sync"
-	"time"
 
 	"github.com/flanksource/canary-checker/api/external"
 	"github.com/flanksource/canary-checker/pkg"
 )
 
-func Error(check external.Check, err error) *pkg.CheckResult {
-	return &pkg.CheckResult{
-		Check:   check,
-		Pass:    false,
-		Invalid: true,
-		Error:   err.Error(),
-	}
-}
-
 func Failf(check external.Check, msg string, args ...interface{}) *pkg.CheckResult {
 	return &pkg.CheckResult{
 		Check:   check,
@@ -27,48 +17,6 @@ func Failf(check external.Check, msg string, args ...interface{}) *pkg.CheckResu
 	}
 }
 
-// TextFailf used for failure in case of text based results
-func TextFailf(check external.Check, textResults bool, msg string, args ...interface{}) *pkg.CheckResult {
-	if textResults {
-		return &pkg.CheckResult{
-			Check:       check,
-			Pass:        false,
-			Invalid:     false,
-			DisplayType: "Text",
-			Message:     fmt.Sprintf(msg, args...),
-		}
-	}
-	return Failf(check, msg, args...)
-}
-func Success(check external.Check, start time.Time) *pkg.CheckResult {
-	return &pkg.CheckResult{
-		Check:    check,
-		Pass:     true,
-		Invalid:  false,
-		Duration: time.Since(start).Milliseconds(),
-	}
-}
-
-func Successf(check external.Check, start time.Time, textResults bool, msg string, args ...interface{}) *pkg.CheckResult {
-	if textResults {
-		return &pkg.CheckResult{
-			Check:       check,
-			Pass:        true,
-			DisplayType: "Text",
-			Invalid:     false,
-			Message:     fmt.Sprintf(msg, args...),
-			Duration:    time.Since(start).Milliseconds(),
-		}
-	}
-	return &pkg.CheckResult{
-		Check:    check,
-		Pass:     true,
-		Invalid:  false,
-		Message:  fmt.Sprintf(msg, args...),
-		Duration: time.Since(start).Milliseconds(),
-	}
-}
-
 func Passf(check external.Check, msg string, args ...interface{}) *pkg.CheckResult {
 	return &pkg.CheckResult{
 		Check:   check,

pkg/results.go

@@ -10,47 +10,20 @@ import (
 
 type Results []*CheckResult
 
-func Fail(check external.Check, canary v1.Canary) *CheckResult {
-	return &CheckResult{
-		Check: check,
-		Data: map[string]interface{}{
-			"results": make(map[string]interface{}),
-		},
-		Start:  time.Now(),
-		Pass:   false,
-		Canary: canary,
-	}
-}
-
-func SetupError(canary v1.Canary, err error) Results {
-	var results Results
-	for _, check := range canary.Spec.GetAllChecks() {
-		results = append(results, &CheckResult{
-			Start:   time.Now(),
-			Pass:    false,
-			Invalid: true,
-			Error:   err.Error(),
-			Check:   check,
-			Data: map[string]interface{}{
-				"results": make(map[string]interface{}),
-			},
-		})
-	}
-	return results
-}
-
 func Invalid(check external.Check, canary v1.Canary, reason string) Results {
 	return Results{&CheckResult{
-		Start: time.Now(),
-		Pass:  false,
-		Error: reason,
-		Check: check,
+		Start:   time.Now(),
+		Pass:    false,
+		Invalid: true,
+		Error:   reason,
+		Check:   check,
 		Data: map[string]interface{}{
 			"results": make(map[string]interface{}),
 		},
 		Canary: canary,
 	}}
 }
+
 func Success(check external.Check, canary v1.Canary) *CheckResult {
 	result := New(check, canary)
 	result.Pass = true
@@ -162,10 +135,12 @@ func (r Results) Failf(msg string, args ...interface{}) Results {
 	r[0].Failf(msg, args...)
 	return r
 }
+
 func (r Results) Invalidf(msg string, args ...interface{}) Results {
 	r[0].Invalidf(msg, args...)
 	return r
 }
+
 func (r Results) WithError(err error) Results {
 	r[0].ErrorObject = err
 	return r