[JENKINS-57841] Change defaults for command and args for Container Te…

…mplate. Document Windows defaults in the help.

src/main/java/org/csanchez/jenkins/plugins/kubernetes/ContainerTemplate.java

@@ -26,6 +26,8 @@
 import jenkins.model.Jenkins;
 import org.kohsuke.stapler.QueryParameter;
 
+import javax.annotation.CheckForNull;
+
 
 public class ContainerTemplate extends AbstractDescribableImpl<ContainerTemplate> implements Serializable {
 
@@ -133,17 +135,19 @@ public void setCommand(String command) {
         this.command = command;
     }
 
+    @CheckForNull
     public String getCommand() {
-        return command;
+        return this.command == null ? DescriptorImpl.defaultCommand : this.command;
     }
 
     @DataBoundSetter
     public void setArgs(String args) {
         this.args = args;
     }
 
+    @CheckForNull
     public String getArgs() {
-        return args;
+        return this.args == null ? DescriptorImpl.defaultArgs : this.args;
     }
 
     @DataBoundSetter
@@ -350,6 +354,8 @@ public FormValidation doCheckImage(@QueryParameter String value) {
         public String getWorkingDir() {
             return DEFAULT_WORKING_DIR;
         }
+        public final static String defaultCommand = "sleep";
+        public final static String defaultArgs = "9999999";
     }
 
     @Override

src/main/resources/org/csanchez/jenkins/plugins/kubernetes/ContainerTemplate/config.jelly

@@ -23,11 +23,11 @@
   </f:entry>
 
   <f:entry field="command" title="${%Command to run}">
-    <f:textbox default="/bin/sh -c"/>
+    <f:textbox default="${descriptor.defaultCommand}"/>
   </f:entry>
 
   <f:entry field="args" title="${%Arguments to pass to the command}">
-    <f:textbox default="cat"/>
+    <f:textbox default="${descriptor.defaultArgs}"/>
   </f:entry>
 
   <f:entry field="ttyEnabled" title="${%Allocate pseudo-TTY}">

src/main/resources/org/csanchez/jenkins/plugins/kubernetes/ContainerTemplate/help-args.html

@@ -1,2 +1,3 @@
-Arguments to pass to the command. 
-${computer.jnlpmac} and ${computer.name} are replaced with the agent secret and name respectively. 
+<p>Arguments to pass to the command.</p>
+<p>${computer.jnlpmac} and ${computer.name} are replaced with the agent secret and name respectively.</p>
+<p>For Windows containers the args ["Start-Sleep", "999999"] are reasonable choices to go with powershell.</p>

src/main/resources/org/csanchez/jenkins/plugins/kubernetes/ContainerTemplate/help-command.html

@@ -1 +1,2 @@
-Override the image entrypoint with a different one
+<p>Override the image entrypoint with a different one.</p>
+<p>For Windows containers 'powershell' is a good default.</p>