From 1707be95288f6427de61d704cab6efc10529bc68 Mon Sep 17 00:00:00 2001
From: Simone Gotti <simone.gotti@gmail.com>
Date: Mon, 29 Jul 2019 09:58:17 +0200
Subject: [PATCH] executor: set task endTime when marking as failed

Add missing set of task endTime when the executor is marking the task as failed
due to no related running task (usually after executor restart).
---
 internal/services/executor/executor.go | 1 +
 1 file changed, 1 insertion(+)

diff --git a/internal/services/executor/executor.go b/internal/services/executor/executor.go
index 5191abe..ed19beb 100644
--- a/internal/services/executor/executor.go
+++ b/internal/services/executor/executor.go
@@ -1162,6 +1162,7 @@ func (e *Executor) taskUpdater(ctx context.Context, et *types.ExecutorTask) {
 		if !ok {
 			log.Infof("marking executor task %s as failed since there's no running task", et.ID)
 			et.Status.Phase = types.ExecutorTaskPhaseFailed
+			et.Status.EndTime = util.TimePtr(time.Now())
 			// mark in progress step as failed too
 			for _, s := range et.Status.Steps {
 				if s.Phase == types.ExecutorTaskPhaseRunning {