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

pom.xml

@@ -94,10 +94,10 @@
  <version>6.8</version>
  </dependency>
 
- <dependency> <!-- OnceRetentionStrategy -->
+ <dependency>
  <groupId>org.jenkins-ci.plugins</groupId>
  <artifactId>durable-task</artifactId>
- <version>1.29</version>
+ <version>1.30-rc416.ace2b55005b1</version> <!-- TODO https://github.com/jenkinsci/durable-task-plugin/pull/95 -->
  </dependency>
  <dependency>
  <groupId>org.jenkins-ci.plugins</groupId>

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

@@ -190,6 +190,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'
+ }
+ }
+}