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
dlorenc · Dec 11, 2020 · ee73bd8 · ee73bd8
1 parent bff0ba0
commit ee73bd8
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 2 deletions.
diff --git a/examples/v1alpha1/taskruns/dind-sidecar.yaml b/examples/v1alpha1/taskruns/dind-sidecar.yaml
@@ -18,7 +18,6 @@ spec:
       - name: DOCKER_CERT_PATH
         value: /certs/client
       script: |
-          #!/usr/bin/env sh
           # Run a Docker container.
           docker run busybox echo hello
 

diff --git a/examples/v1beta1/taskruns/dind-sidecar.yaml b/examples/v1beta1/taskruns/dind-sidecar.yaml
@@ -18,7 +18,6 @@ spec:
       - name: DOCKER_CERT_PATH
         value: /certs/client
       script: |
-          #!/usr/bin/env sh
           # Run a Docker container.
           docker run busybox echo hello