fix: remove defer to certainly save the startTime

t-kikuc · t-kikuc · commit f71c4d4f5132 · 2024-12-27T17:02:39.000+09:00
Signed-off-by: t-kikuc &lt;tkikuchi07f@gmail.com&gt;
diff --git a/pkg/app/pipedv1/plugin/wait/deployment/wait.go b/pkg/app/pipedv1/plugin/wait/deployment/wait.go
@@ -48,7 +48,7 @@ func (s *deploymentServiceServer) execute(ctx context.Context, in *deployment.Ex
 		// When this is the first run.
 		initialStart = time.Now()
 	}
-	defer s.saveStartTime(ctx, initialStart, in.Stage.Id)
+	s.saveStartTime(ctx, initialStart, in.Stage.Id)
 
 	return wait(ctx, duration, initialStart, slp)
 }

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ func (s deploymentServiceServer) execute(ctx context.Context, in deployment.Ex`
`48`	`48`	`// When this is the first run.`
`49`	`49`	`initialStart = time.Now()`
`50`	`50`	`}`
`51`		`- defer s.saveStartTime(ctx, initialStart, in.Stage.Id)`
	`51`	`+ s.saveStartTime(ctx, initialStart, in.Stage.Id)`
`52`	`52`
`53`	`53`	`return wait(ctx, duration, initialStart, slp)`
`54`	`54`	`}`