Fix: Pass -e to sh and bash during command execution (#3065)

## Changes
This PR fixes a regression introduced in
#2896 where we stopped passing the
`-e` flag to the shell, which makes both sh and bash bail out at the
first command with a non-zero exit code.

## Why
Adding back `-e` keeps the behaviour consistent with the one before
#2896 was introduced.

## Tests
Newly added regression tests

Author: shreyas-goenka

.wsignore

@@ -17,6 +17,10 @@ experimental/python/docs/images/databricks-logo.svg
 # In order to prevent that difference, hello.txt does not have a trailing newline.
 acceptance/selftest/record_cloud/volume-io/hello.txt
 
+# error output when artifact build fails contains trailing whitespace:
+acceptance/bundle/artifacts/shell/err-bash/output.txt
+acceptance/bundle/artifacts/shell/err-sh/output.txt
+
 # "bundle init" has trailing whitespace:
 acceptance/bundle/templates-machinery/helpers-error/output.txt
 

acceptance/bundle/artifacts/shell/err-bash/databricks.yml

@@ -0,0 +1,8 @@
+artifacts:
+  my_artifact:
+    executable: bash
+    build: |-
+      echo "hello" > out.shell.txt
+      echo "world" >> out.shell.txt
+      false
+      echo "this should not be printed since bash is run with -e" >> out.shell.txt

acceptance/bundle/artifacts/shell/err-bash/out.shell.txt

@@ -0,0 +1,2 @@
+hello
+world

acceptance/bundle/artifacts/shell/err-bash/output.txt

@@ -0,0 +1,7 @@
+
+>>> [CLI] bundle deploy
+Building my_artifact...
+Error: build failed my_artifact, error: exit status 1, output: 
+
+
+Exit code: 1

acceptance/bundle/artifacts/shell/err-bash/script

@@ -0,0 +1 @@
+trace $CLI bundle deploy

acceptance/bundle/artifacts/shell/err-sh/databricks.yml

@@ -0,0 +1,8 @@
+artifacts:
+  my_artifact:
+    executable: sh
+    build: |-
+      echo "hello" > out.shell.txt
+      echo "world" >> out.shell.txt
+      false
+      echo "this should not be printed since sh is run with -e" >> out.shell.txt

acceptance/bundle/artifacts/shell/err-sh/out.shell.txt

@@ -0,0 +1,2 @@
+hello
+world

acceptance/bundle/artifacts/shell/err-sh/output.txt

@@ -0,0 +1,7 @@
+
+>>> [CLI] bundle deploy
+Building my_artifact...
+Error: build failed my_artifact, error: exit status 1, output: 
+
+
+Exit code: 1

acceptance/bundle/artifacts/shell/err-sh/script

@@ -0,0 +1 @@
+trace $CLI bundle deploy

libs/exec/shell_bash.go

@@ -13,7 +13,7 @@ type bashShell struct {
 func (s bashShell) prepare(command string) (*execContext, error) {
 	return &execContext{
 		executable: s.executable,
-		args:       []string{"-c", command},
+		args:       []string{"-ec", command},
 	}, nil
 }