fix: use scheduled at instead of current time for empty jobrun data

kushsharma · kushsharma · commit c36818bb4fe8 · 2022-01-13T11:42:40.000+05:30
Signed-off-by: Kush &lt;thekushsharma@gmail.com&gt;
diff --git a/store/postgres/adapter.go b/store/postgres/adapter.go
@@ -494,7 +494,7 @@ func (adapt JobSpecAdapter) ToJobRun(jr JobRun) (models.JobRun, models.Namespace
 		// to make it backward compatible, generate execution time
 		// although this time may not match exactly what it should be
 		// but will avoid failing
-		adaptedData.ExecutedAt = time.Now().UTC()
+		adaptedData.ExecutedAt = jr.ScheduledAt
 	}
 
 	var instanceSpecs []models.InstanceSpec

Original file line number	Diff line number	Diff line change
`@@ -494,7 +494,7 @@ func (adapt JobSpecAdapter) ToJobRun(jr JobRun) (models.JobRun, models.Namespace`
`494`	`494`	`// to make it backward compatible, generate execution time`
`495`	`495`	`// although this time may not match exactly what it should be`
`496`	`496`	`// but will avoid failing`
`497`		`- adaptedData.ExecutedAt = time.Now().UTC()`
	`497`	`+ adaptedData.ExecutedAt = jr.ScheduledAt`
`498`	`498`	`}`
`499`	`499`
`500`	`500`	`var instanceSpecs []models.InstanceSpec`