chore: return explicit error on empty schedule

flanksource · Jan 31, 2024 · 2d82dd9 · 2d82dd9
1 parent b1f95bc
commit 2d82dd9
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/job/job.go b/job/job.go
@@ -394,6 +394,11 @@ func (j *Job) AddToScheduler(cronRunner *cron.Cron) error {
 	if override, ok := getProperty(j, j.Context.Properties(), "schedule"); ok {
 		schedule = override
 	}
+
+	if schedule == "" {
+		return fmt.Errorf("job schedule cannot be empty")
+	}
+
 	if schedule == "@never" {
 		j.Context.Infof("skipping scheduling")
 		return nil