Remove 0s timeout as this is the default but causing failures

gabemontero · Oct 8, 2021 · 7d2f69b · 7d2f69b
1 parent 7e33efe
commit 7d2f69b
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 0 deletions.
diff --git a/pkg/shp/flags/build.go b/pkg/shp/flags/build.go
@@ -57,4 +57,7 @@ func SanitizeBuildSpec(b *buildv1alpha1.BuildSpec) {
 			b.Builder = nil
 		}
 	}
+	if b.Timeout != nil && b.Timeout.Duration == 0 {
+		b.Timeout = nil
+	}
 }
diff --git a/pkg/shp/flags/build_test.go b/pkg/shp/flags/build_test.go
@@ -155,6 +155,12 @@ func TestSanitizeBuildSpec(t *testing.T) {
 		name: "should not clean-up complete objects",
 		in:   completeBuildSpec,
 		out:  completeBuildSpec,
+	}, {
+		name: "should clean-up 0s duration",
+		in: buildv1alpha1.BuildSpec{Timeout: &metav1.Duration{
+			Duration: time.Duration(0),
+		}},
+		out: buildv1alpha1.BuildSpec{Timeout: nil},
 	}}
 
 	for _, tt := range testCases {

diff --git a/pkg/shp/flags/buildrun.go b/pkg/shp/flags/buildrun.go
@@ -49,4 +49,7 @@ func SanitizeBuildRunSpec(br *buildv1alpha1.BuildRunSpec) {
 			br.Output = nil
 		}
 	}
+	if br.Timeout != nil && br.Timeout.Duration == 0 {
+		br.Timeout = nil
+	}
 }
diff --git a/pkg/shp/flags/buildrun_test.go b/pkg/shp/flags/buildrun_test.go
@@ -107,6 +107,12 @@ func TestSanitizeBuildRunSpec(t *testing.T) {
 		name: "should not clean-up complete objects",
 		in:   completeBuildRunSpec,
 		out:  completeBuildRunSpec,
+	}, {
+		name: "should clean-up 0s duration",
+		in: buildv1alpha1.BuildRunSpec{Timeout: &metav1.Duration{
+			Duration: time.Duration(0),
+		}},
+		out: buildv1alpha1.BuildRunSpec{Timeout: nil},
 	}}
 
 	for _, tt := range testCases {