Mii app patching test (#1616)

* Initial changes to test application update

* Add app assertions

* Save the current change

* Update ItMiiDomain test

* More changes

* verify an app from a server pod

* add code to patch the domain with a new image and verify

* Revert unintended changes

* Fix merge issues

* more cleanup

* Initial change for test number 6

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

* Initial change for adding another app

* More impl for the second test case

* Add model file for the second app

* Minor cleanup

* Clean up comments

* Minor fix

* Move delete images to the end of AfterAll

* Update command to check response

* Fix a merge error

* Fix exec command

* Add code for remote repo stuff for the new tests

* Minor fixes

* Cleanup

* Fix image name

* Add an app file

* Make sure the thread joins before we save the stdout

* Remove ItWitValidation class becuase the test in it has been copied to ItMiiDomain class

* Revert "Remove ItWitValidation class becuase the test in it has been copied to ItMiiDomain class"

This reverts commit c243b49.

* Debugging

* Fix pushed image name

* Turn off debugging

* Testing

* Change single check of expected condition to quick retry

* Check app on all managed servers

* Check each pod to verify if the image has been patched

* Fine tuning

* clean up

* More cleanup

* Continue work in progress

* Add app continuous availability check and enable it with a system property

* Clean up

* More javadoc

* javadoc and cleanup

* Adjust timeout values

* Fine tuning

* Minor fix

* Use kubectl exec command for checking app continuous availability

* clean up javadoc

* Cleanup

* Minor update

* Minor update of log messages

* More cleanup of log messages and comments

* Address review comments

* Address one more comment

* Minor fix

* More logging

* Minor comment update

* Make log messages more informative

* Address a new review comment

* Fix a minor merge error

* Address more review comments

* Update javadoc

* Fix merge issues

* Update the push image code to check REPO_NAME

* remove double adding REMO_NAME

* Change baseImageName to image Name

* Fix a typo

Author: doxiao

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

@@ -16,7 +16,6 @@
 import io.kubernetes.client.openapi.models.V1EnvVar;
 import io.kubernetes.client.openapi.models.V1LocalObjectReference;
 import io.kubernetes.client.openapi.models.V1ObjectMeta;
-import io.kubernetes.client.openapi.models.V1Pod;
 import io.kubernetes.client.openapi.models.V1Secret;
 import io.kubernetes.client.openapi.models.V1SecretReference;
 import io.kubernetes.client.openapi.models.V1ServiceAccount;
@@ -497,21 +496,14 @@ private void checkServiceCreated(String serviceName, String domNamespace) {
   }
 
   private void checkServerReadyStatusByExec(String podName, String namespace) {
-    final V1Pod pod = assertDoesNotThrow(() -> oracle.weblogic.kubernetes.assertions.impl.Kubernetes
-        .getPod(namespace, null, podName));
-
-    if (pod != null) {
-      ExecResult execResult = assertDoesNotThrow(
-          () -> execCommand(pod, null, true, READ_STATE_COMMAND));
-      if (execResult.exitValue() == 0) {
-        logger.info("execResult: " + execResult);
-        assertEquals("RUNNING", execResult.stdout(),
-            "Expected " + podName + ", in namespace " + namespace + ", to be in RUNNING ready status");
-      } else {
-        fail("Ready command failed with exit status code: " + execResult.exitValue());
-      }
+    ExecResult execResult = assertDoesNotThrow(
+        () -> execCommand(namespace, podName, null, true, READ_STATE_COMMAND));
+    if (execResult.exitValue() == 0) {
+      logger.info("execResult: " + execResult);
+      assertEquals("RUNNING", execResult.stdout(),
+          "Expected " + podName + ", in namespace " + namespace + ", to be in RUNNING ready status");
     } else {
-      fail("Did not find pod " + podName + " in namespace " + namespace);
+      fail("Ready command failed with exit status code: " + execResult.exitValue());
     }
   }
 

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

@@ -18,7 +18,6 @@
 import io.kubernetes.client.openapi.models.V1EnvVar;
 import io.kubernetes.client.openapi.models.V1LocalObjectReference;
 import io.kubernetes.client.openapi.models.V1ObjectMeta;
-import io.kubernetes.client.openapi.models.V1Pod;
 import io.kubernetes.client.openapi.models.V1Secret;
 import io.kubernetes.client.openapi.models.V1SecretReference;
 import io.kubernetes.client.openapi.models.V1ServiceAccount;
@@ -566,21 +565,14 @@ private void checkServiceCreated(String serviceName, String domNamespace) {
   }
 
   private void checkServerReadyStatusByExec(String podName, String namespace) {
-    final V1Pod pod = assertDoesNotThrow(() -> oracle.weblogic.kubernetes.assertions.impl.Kubernetes
-        .getPod(namespace, null, podName));
-
-    if (pod != null) {
-      ExecResult execResult = assertDoesNotThrow(
-          () -> execCommand(pod, null, true, READ_STATE_COMMAND));
-      if (execResult.exitValue() == 0) {
-        logger.info("execResult: " + execResult);
-        assertEquals("RUNNING", execResult.stdout(),
-            "Expected " + podName + ", in namespace " + namespace + ", to be in RUNNING ready status");
-      } else {
-        fail("Ready command failed with exit status code: " + execResult.exitValue());
-      }
+    ExecResult execResult = assertDoesNotThrow(
+        () -> execCommand(namespace, podName, null, true, READ_STATE_COMMAND));
+    if (execResult.exitValue() == 0) {
+      logger.info("execResult: " + execResult);
+      assertEquals("RUNNING", execResult.stdout(),
+          "Expected " + podName + ", in namespace " + namespace + ", to be in RUNNING ready status");
     } else {
-      fail("Did not find pod " + podName + " in namespace " + namespace);
+      fail("Ready command failed with exit status code: " + execResult.exitValue());
     }
   }