minor fixes

armadaproject · Jan 20, 2025 · e5ae56a · e5ae56a
1 parent 6ec31a1
commit e5ae56a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/cmd/scheduleringester/main.go b/cmd/scheduleringester/main.go
@@ -4,14 +4,12 @@ import (
 	"fmt"
 	"os"
 
-	"github.com/armadaproject/armada/internal/common/logging"
-
 	"github.com/spf13/pflag"
 	"github.com/spf13/viper"
 
-	"github.com/armadaproject/armada/internal/scheduleringester"
-
 	"github.com/armadaproject/armada/internal/common"
+	"github.com/armadaproject/armada/internal/common/logging"
+	"github.com/armadaproject/armada/internal/scheduleringester"
 )
 
 const CustomConfigLocation string = "config"

diff --git a/internal/executor/application.go b/internal/executor/application.go
@@ -19,7 +19,6 @@ import (
 	"github.com/armadaproject/armada/internal/common/cluster"
 	"github.com/armadaproject/armada/internal/common/etcdhealth"
 	"github.com/armadaproject/armada/internal/common/healthmonitor"
-	log "github.com/armadaproject/armada/internal/common/logging"
 	common_metrics "github.com/armadaproject/armada/internal/common/metrics"
 	"github.com/armadaproject/armada/internal/common/task"
 	"github.com/armadaproject/armada/internal/common/util"
@@ -195,8 +194,7 @@ func setupExecutorApiComponents(
 
 	failedPodChecker, err := failedpodchecks.NewPodRetryChecker(config.Kubernetes.FailedPodChecks)
 	if err != nil {
-		log.Errorf("Failed to create job event reporter: %s", err)
-		os.Exit(-1)
+		ctx.Fatalf("Config error in failed pod checks: %s", err)
 	}
 
 	eventReporter, stopReporter := reporter.NewJobEventReporter(eventSender, clock.RealClock{}, 200)