Merge pull request #1033 from Vlatombe/revert-getLogFile

Author: Vlatombe

src/test/java/org/csanchez/jenkins/plugins/kubernetes/pipeline/KubernetesPipelineTest.java

@@ -41,7 +41,6 @@
 import static org.junit.Assert.assertTrue;
 import static org.junit.Assume.*;
 
-import java.io.File;
 import java.util.Arrays;
 import java.util.Collections;
 import java.util.List;
@@ -154,14 +153,8 @@ public void runInPod() throws Exception {
             new Thread(() -> {
                 long pos = 0;
                 try {
-                    while (true) {
-                        Jenkins j = Jenkins.getInstanceOrNull();
-                        if (j == null) {
-                            break;
-                        }
-                        // not using Computer#getLogDir as it has side-effect of creating log directories which can create a race condition with main code
-                        File logFile = new File(j.getRootDir(), "logs/slaves/" + c.getName() + "/slave.log");
-                        if (logFile.isFile()) { // TODO should LargeText.FileSession handle this?
+                    while (Jenkins.getInstanceOrNull() != null) { // otherwise get NPE from Computer.getLogDir
+                        if (c.getLogFile().isFile()) { // TODO should LargeText.FileSession handle this?
                             pos = c.getLogText().writeLogTo(pos, System.out);
                         }
                         Thread.sleep(100);