Skip to content

Commit

Permalink
Set explicit parallelism for e2e tests, increase Kaniko e2e test time…
Browse files Browse the repository at this point in the history
…outs

fixes #4863

The parallelism change appears to make a real difference in cutting down on flaky
tests, while only increasing total duration of the integration test runs to ~30-35
minutes from ~25, which seems like a worthy trade to me.

Also bumps the timeout for Kaniko tasks in e2e tests to 5 minutes, where a timeout
was specified in the first place.

Signed-off-by: Andrew Bayer <[email protected]>
  • Loading branch information
abayer committed May 17, 2022
1 parent c938ee5 commit 895582f
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion test/e2e-tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ function run_e2e() {
# and they cause a lot of noise in the logs, making it harder to debug integration
# test failures.
if [ "${RUN_YAML_TESTS}" == "true" ]; then
go_test_e2e -mod=readonly -tags=examples -timeout=20m ./test/ || failed=1
go_test_e2e -parallel=4 -mod=readonly -tags=examples -timeout=20m ./test/ || failed=1
fi
}

Expand Down
2 changes: 1 addition & 1 deletion test/kaniko_task_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -199,7 +199,7 @@ metadata:
spec:
taskRef:
name: %s
timeout: 2m
timeout: 5m
resources:
inputs:
- name: gitsource
Expand Down
2 changes: 1 addition & 1 deletion test/v1alpha1/kaniko_task_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -193,7 +193,7 @@ metadata:
spec:
taskRef:
name: %s
timeout: 2m
timeout: 5m
inputs:
resources:
- name: gitsource
Expand Down

0 comments on commit 895582f

Please sign in to comment.