chore: use consts

kubeshop · Aug 13, 2024 · 95a1911 · 95a1911
1 parent 5652bfd
commit 95a1911
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/pkg/testworkflows/testworkflowcontroller/podstate.go b/pkg/testworkflows/testworkflowcontroller/podstate.go
@@ -495,7 +495,7 @@ func initializePodState(parentCtx context.Context, pod Channel[*corev1.Pod], pod
 						} else if ok {
 							state.RegisterPod(p.Value)
 						}
-					case <-time.After(2 * time.Second):
+					case <-time.After(alignmentTimeout):
 						// Continue - likely we won't receive Pod status
 					}
 				}

diff --git a/pkg/testworkflows/testworkflowcontroller/watchinstrumentedpod.go b/pkg/testworkflows/testworkflowcontroller/watchinstrumentedpod.go
@@ -22,8 +22,9 @@ import (
 )
 
 const (
-	InitContainerName = "tktw-init"
-	IdleTimeout       = 100 * time.Millisecond
+	InitContainerName    = "tktw-init"
+	IdleTimeout          = 100 * time.Millisecond
+	ExpectedDelayTimeout = 1 * time.Second
 )
 
 type WatchInstrumentedPodOptions struct {
@@ -80,7 +81,7 @@ func WatchInstrumentedPod(parentCtx context.Context, clientSet kubernetes.Interf
 		// Handle when the execution have been finished, but there may be no Pod
 		case <-state.Finished(""):
 			select {
-			case <-time.After(1 * time.Second):
+			case <-time.After(ExpectedDelayTimeout):
 				s.Error(fmt.Errorf("the execution is finished, but failed to get pod"))
 				return
 			case p := <-pod.Peek(ctx):