Improve backend schedule logs

smarthome-go · Sep 11, 2022 · fb736d3 · fb736d3
1 parent 33078d0
commit fb736d3
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/core/scheduler/scheduler/runner.go b/core/scheduler/scheduler/runner.go
@@ -42,11 +42,8 @@ func scheduleRunnerFunc(id uint) {
 		return
 	}
 	if !found {
-		log.Warn("Schedule failed because owner user does not exist anymore, deleting schedule...")
-		if err := database.DeleteScheduleById(id); err != nil {
-			log.Error("Cleaning up dangling schedule failed: could not remove schedule from database: ", err.Error())
-			return
-		}
+		log.Warn("Schedule failed because owner user does not exist anymore, skipping execution...")
+		return
 	}
 	if !owner.SchedulerEnabled {
 		log.Debug(fmt.Sprintf("Schedule '%s' was not executed because its owner has disabled their schedules & automations", job.Data.Name))
@@ -69,7 +66,7 @@ func scheduleRunnerFunc(id uint) {
 		)
 		return
 	}
-	log.Debug(fmt.Sprintf("Schedule '%d' is executing its target", id))
+	log.Debug(fmt.Sprintf("Schedule '%s' (%d) is executing...", job.Data.Name, id))
 	switch job.Data.TargetMode {
 	case database.ScheduleTargetModeCode:
 		_, _, _, hmsErrors := homescript.HmsManager.Run(