Merge pull request #45 from raydouglass/fix-side-container-network

Remove double network for side containers

Author: raydouglass

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

@@ -52,7 +52,9 @@ private DockerNetwork(String id) {
      */
     public static DockerNetwork create(AbstractDockerLauncher launcher) throws IOException, InterruptedException {
         ArgumentListBuilder args = new ArgumentListBuilder();
-        args.add("docker", "network", "create", UUID.randomUUID().toString());
+        args.add("docker", "network", "create", "-d", "bridge",
+                 UUID.randomUUID().toString());
+
         ByteArrayOutputStream baos = new ByteArrayOutputStream();
         int status = launcher.executeCommand(args)
                 .stdout(baos)

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

@@ -249,11 +249,14 @@ private static ArgumentListBuilder getlaunchArgs(RemoteDockerBuildWrapper buildW
 
         //TODO Set name? Maybe with build.toString().replaceAll("^\\w", "_")
         ArgumentListBuilder args = new ArgumentListBuilder()
-                .add("run", "-t", "-d")
-                //Add bridge network for internet access
-                .add("--network", "bridge");
-        //Add inter-container network if needed
-        network.ifPresent(net -> net.addArgs(args));
+                .add("run", "-t", "-d");
+        if (network.isPresent()) {
+            //Add inter-container network
+            network.get().addArgs(args);
+        } else {
+            //Add bridge network for internet access
+            args.add("--network", "bridge");
+        }
 
         if (isMain) {
             String secondaryTempPath = WorkspaceList.tempDir(workspace)