Fix command stdout (#1587)

* Fix ExecCommand to allow both redirect and saveResults enabled the same time

* minor cleanup

Author: doxiao

new-integration-tests/src/test/java/oracle/weblogic/kubernetes/utils/ExecCommand.java

@@ -64,8 +64,13 @@ public static ExecResult exec(
     Thread out = null;
 
     try {
+      String stdout = null;
       if (isRedirectToOut) {
         InputStream i = in.getInputStream();
+        i.mark(0);
+        // read the stdout first, and reset before we redirect the output
+        stdout = read(i);
+        i.reset();
         @SuppressWarnings("resource")
         CopyingOutputStream copyOut = new CopyingOutputStream(System.out);
         // this makes sense because CopyingOutputStream is an InputStreamWrapper
@@ -83,7 +88,13 @@ public static ExecResult exec(
       }
 
       p.waitFor();
-      return new ExecResult(p.exitValue(), read(in.getInputStream()), read(p.getErrorStream()));
+      
+      // if we have not read the stdout, we do it now
+      if (stdout == null) {
+        stdout = read(in.getInputStream());
+      }
+      return new ExecResult(p.exitValue(), stdout, read(p.getErrorStream()));
+    
     } finally {
       if (out != null) {
         out.join();