Merge pull request #53380 from soltysh/issue35507

Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Increase backoffLimit for job that we expect to fail several times **What this PR does / why we need it**: Since the introduction of `backoffLimit` for a job that single test failed majority of times on: `BackoffLimitExceeded: Job has reach the specified backoff limit`. I'm bumping this to 999, so that it has enough room to fail several times. **Which issue this PR fixes**: Fixes #35507. **Special notes for your reviewer**: **Release note**: ```release-note None ```

Merge pull request #53380 from soltysh/issue35507
1797255c · Kubernetes Submit Queue · GitHub · 3ac4762d · 9991a888 · 1797255c
Commit 1797255c authored Oct 10, 2017 by Kubernetes Submit Queue Committed by GitHub Oct 10, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

job.go test/e2e/apps/job.go +1 -1

No files found.
--- a/test/e2e/apps/job.go
+++ b/test/e2e/apps/job.go
@@ -77,7 +77,7 @@ var _ = SIGDescribe("Job", func() {
 		// Worst case analysis: 15 failures, each taking 1 minute to
 		// run due to some slowness, 1 in 2^15 chance of happening,
 		// causing test flake.  Should be very rare.
-		job := framework.NewTestJob("randomlySucceedOrFail", "rand-non-local", v1.RestartPolicyNever, parallelism, completions, nil, backoffLimit)
+		job := framework.NewTestJob("randomlySucceedOrFail", "rand-non-local", v1.RestartPolicyNever, parallelism, completions, nil, 999)
 		job, err := framework.CreateJob(f.ClientSet, f.Namespace.Name, job)
 		Expect(err).NotTo(HaveOccurred())