Skip to content

Commit e9255ae

Browse files
author
Dan Lorenc
committed
Add a "set -e" to the dind-sidecar test script.
I was bitten by this test not having one, when the "docker build" failed but we continued on executing anyway. I did a cursory look at the other tests, and they either already have this or are so short/simple that it doesn't matter. Ref tektoncd#3628
1 parent bff0ba0 commit e9255ae

File tree

2 files changed

+2
-0
lines changed

2 files changed

+2
-0
lines changed

examples/v1alpha1/taskruns/dind-sidecar.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ spec:
1919
value: /certs/client
2020
script: |
2121
#!/usr/bin/env sh
22+
set -x
2223
# Run a Docker container.
2324
docker run busybox echo hello
2425

examples/v1beta1/taskruns/dind-sidecar.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ spec:
1919
value: /certs/client
2020
script: |
2121
#!/usr/bin/env sh
22+
set -x
2223
# Run a Docker container.
2324
docker run busybox echo hello
2425

0 commit comments

Comments
 (0)