- fix process detection for windows container

Author: Stefan Lengauer

src/main/java/org/jenkinsci/plugins/docker/workflow/client/DockerClient.java

@@ -140,7 +140,14 @@ public String run(@Nonnull EnvVars launchEnv, @Nonnull String image, @CheckForNu
     }
 
     public List<String> listProcess(@Nonnull EnvVars launchEnv, @Nonnull String containerId) throws IOException, InterruptedException {
-        LaunchResult result = launch(launchEnv, false, "top", containerId, "-eo", "pid,comm");
+        LaunchResult result;
+        boolean iswin = !launcher.isUnix();
+        if (iswin){
+            result = launch(launchEnv, false, "top", containerId);
+        } else {
+            // comm,pid so that the output matches the windows columns
+            result = launch(launchEnv, false, "top", containerId, "-eo", "comm,pid");
+        }
         if (result.getStatus() != 0) {
             throw new IOException(String.format("Failed to run top '%s'. Error: %s", containerId, result.getErr()));
         }
@@ -154,8 +161,8 @@ public List<String> listProcess(@Nonnull EnvVars launchEnv, @Nonnull String cont
                 if (stringTokenizer.countTokens() < 2) {
                     throw new IOException("Unexpected `docker top` output : "+line);
                 }
-                stringTokenizer.nextToken(); // PID
-                processes.add(stringTokenizer.nextToken()); // COMMAND
+                // Windows containers are started without the .exe suffix to cat but top returns cat.exe
+                processes.add(iswin ? stringTokenizer.nextToken().replace(".exe","") : stringTokenizer.nextToken()); // COMMAND
             }
         }
         return processes;