FIX Fixes several bugs

Author: raydouglass

pom.xml

@@ -12,7 +12,7 @@
 
     <groupId>com.gpuopenanalytics.jenkins</groupId>
     <artifactId>remote-docker-plugin</artifactId>
-    <version>0.0.1-SNAPSHOT</version>
+    <version>0.0.2-SNAPSHOT</version>
     <packaging>hpi</packaging>
 
     <name>Remote Docker Build</name>

src/main/java/com/gpuopenanalytics/jenkins/remotedocker/DockerLauncher.java

@@ -159,7 +159,6 @@ private ArgumentListBuilder getlaunchArgs(DockerConfiguration config,
         //TODO Set name? Maybe with build.toString().replaceAll("^\\w", "_")
         ArgumentListBuilder args = new ArgumentListBuilder()
                 .add("run", "-t", "-d")
-                .add("--env", "TMPDIR=" + workspacePath + ".tmp")
                 .add("--workdir", workspacePath)
                 //Add bridge network for internet access
                 .add("--network", "bridge");

src/main/java/com/gpuopenanalytics/jenkins/remotedocker/config/EnvironmentVariableConfigItem.java

@@ -51,6 +51,10 @@ public void validate() throws Descriptor.FormException {
         //no-op
     }
 
+    public String getEnvironment() {
+        return environment;
+    }
+
     @Override
     public void addCreateArgs(DockerLauncher launcher,
                               ArgumentListBuilder args,

src/main/java/com/gpuopenanalytics/jenkins/remotedocker/config/ExtraDockerArgsConfigItem.java

@@ -49,6 +49,10 @@ public ExtraDockerArgsConfigItem(String extraArgs) {
         this.extraArgs = extraArgs;
     }
 
+    public String getExtraArgs() {
+        return extraArgs;
+    }
+
     @Override
     public void validate() throws Descriptor.FormException {
 

src/main/java/com/gpuopenanalytics/jenkins/remotedocker/config/MemoryConfigItem.java

@@ -50,6 +50,10 @@ public MemoryConfigItem(String memory) {
         this.memory = memory;
     }
 
+    public String getMemory() {
+        return memory;
+    }
+
     @Override
     public void validate() throws Descriptor.FormException {
         if (!Utils.hasVariablesToResolve(memory)) {

src/main/resources/com/gpuopenanalytics/jenkins/remotedocker/RemoteDockerBuildWrapper/config.jelly

@@ -37,5 +37,8 @@
             <f:repeatableDeleteButton/>
         </f:repeatableProperty>
     </f:entry>
+    <f:entry title="Debug" field="debug">
+        <f:checkbox/>
+    </f:entry>
 
 </j:jelly>

src/main/resources/com/gpuopenanalytics/jenkins/remotedocker/config/DockerRuntimeConfigItem/config.jelly

@@ -25,7 +25,7 @@
 
 <j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define" xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form">
 
-    <f:radioBlock name="dockerRuntime" value="runc" title="runc - Standard Docker runtime (default)" inline="true" checked="${instance.getValue()=='runc'}" />
+    <f:radioBlock name="dockerRuntime" value="runc" title="runc - Standard Docker runtime (default)" inline="true" checked="${instance.getValue()=='runc' or empty(instance.getValue())}" />
     <f:radioBlock name="dockerRuntime" value="nvidia" title="nvidia - Enable NVIDIA GPU support with nvidia-docker" inline="true" checked="${instance.getValue()=='nvidia'}" />
     <f:radioBlock name="dockerRuntime" value="custom" title="custom - Define which runtime to use" inline="true" checked="${instance.isCustom()}">
         <f:entry title="Custom runtime to use" field="dockerRuntimeCustom">

src/main/resources/com/gpuopenanalytics/jenkins/remotedocker/job/AbstractDockerConfiguration/config.jelly

@@ -32,10 +32,6 @@
     <f:advanced>
         <f:section title="Advanced Settings">
 
-            <f:entry title="Debug" field="debug">
-                <f:checkbox/>
-            </f:entry>
-
             <st:include page="advanced-config.jelly" class="${descriptor.clazz}" optional="true"/>
 
             <f:entry title="Additional Settings">