Merge pull request #490 from jglick/bourneShellElsewhereInPath

Verifying that tools do not assume sh is in /bin/sh

src/main/java/org/csanchez/jenkins/plugins/kubernetes/pipeline/ContainerExecDecorator.java

@@ -79,7 +79,7 @@ public class ContainerExecDecorator extends LauncherDecorator implements Seriali
     private static final String COOKIE_VAR = "JENKINS_SERVER_COOKIE";
 
     private static final Logger LOGGER = Logger.getLogger(ContainerExecDecorator.class.getName());
-    private static final String DEFAULT_SHELL="/bin/sh";
+    private static final String DEFAULT_SHELL = "sh";
 
     /**
      * stdin buffer size for commands sent to Kubernetes exec api. A low value will cause slowness in commands executed.

src/test/java/org/csanchez/jenkins/plugins/kubernetes/pipeline/KubernetesPipelineTest.java

@@ -198,6 +198,12 @@ public void runInPodWithDifferentShell() throws Exception {
         */
     }
 
+    @Test
+    public void bourneShellElsewhereInPath() throws Exception {
+        r.assertBuildStatusSuccess(r.waitForCompletion(b));
+        r.assertLogContains("/kaniko:/busybox", b);
+    }
+
     @Test
     public void runInPodWithMultipleContainers() throws Exception {
         r.assertBuildStatusSuccess(r.waitForCompletion(b));

src/test/resources/org/csanchez/jenkins/plugins/kubernetes/pipeline/bourneShellElsewhereInPath.groovy

@@ -0,0 +1,7 @@
+podTemplate(label: '$NAME', containers: [containerTemplate(name: 'kaniko', image: 'gcr.io/kaniko-project/executor:debug', command: 'sleep', args: '99d')]) {
+    node('$NAME') {
+        container(name: 'kaniko') {
+            sh 'echo $PATH'
+        }
+    }
+}