feat: allow to skip namespace deletion in junit extension

This is important for tests which are using KubeApiTest from fabric8 client.
There is no namespace controller present, so namespaces cannot be deleted.
Although waitForNamespaceDeletion already is a workaround, but thought might be
nicer to have this more explicit.

Signed-off-by: Attila Mészáros <a_meszaros@apple.com>

Author: csviri

operator-framework-junit/src/main/java/io/javaoperatorsdk/operator/junit/AbstractOperatorExtension.java

@@ -58,6 +58,7 @@ public abstract class AbstractOperatorExtension
   protected Duration infrastructureTimeout;
   protected final boolean oneNamespacePerClass;
   protected final boolean preserveNamespaceOnError;
+  protected final boolean skipNamespaceDeletion;
   protected final boolean waitForNamespaceDeletion;
   protected final int namespaceDeleteTimeout = DEFAULT_NAMESPACE_DELETE_TIMEOUT;
   protected final Function<ExtensionContext, String> namespaceNameSupplier;
@@ -70,6 +71,7 @@ protected AbstractOperatorExtension(
       Duration infrastructureTimeout,
       boolean oneNamespacePerClass,
       boolean preserveNamespaceOnError,
+      boolean skipNamespaceDeletion,
       boolean waitForNamespaceDeletion,
       KubernetesClient kubernetesClient,
       KubernetesClient infrastructureKubernetesClient,
@@ -85,6 +87,7 @@ protected AbstractOperatorExtension(
     this.infrastructureTimeout = infrastructureTimeout;
     this.oneNamespacePerClass = oneNamespacePerClass;
     this.preserveNamespaceOnError = preserveNamespaceOnError;
+    this.skipNamespaceDeletion = skipNamespaceDeletion;
     this.waitForNamespaceDeletion = waitForNamespaceDeletion;
     this.namespaceNameSupplier = namespaceNameSupplier;
     this.perClassNamespaceNameSupplier = perClassNamespaceNameSupplier;
@@ -201,6 +204,9 @@ protected void after(ExtensionContext context) {
     if (namespace != null) {
       if (preserveNamespaceOnError && context.getExecutionException().isPresent()) {
         LOGGER.info("Preserving namespace {}", namespace);
+      } else if (skipNamespaceDeletion) {
+        LOGGER.info("Skipping namespace deletion for {}", namespace);
+        deleteOperator();
       } else {
         infrastructureKubernetesClient.resourceList(infrastructure).delete();
         deleteOperator();
@@ -229,6 +235,7 @@ public abstract static class AbstractBuilder<T extends AbstractBuilder<T>> {
     protected final List<HasMetadata> infrastructure;
     protected Duration infrastructureTimeout;
     protected boolean preserveNamespaceOnError;
+    protected boolean skipNamespaceDeletion;
     protected boolean waitForNamespaceDeletion;
     protected boolean oneNamespacePerClass;
     protected int namespaceDeleteTimeout;
@@ -245,6 +252,9 @@ protected AbstractBuilder() {
       this.preserveNamespaceOnError =
           Utils.getSystemPropertyOrEnvVar("josdk.it.preserveNamespaceOnError", false);
 
+      this.skipNamespaceDeletion =
+          Utils.getSystemPropertyOrEnvVar("josdk.it.skipNamespaceDeletion", false);
+
       this.waitForNamespaceDeletion =
           Utils.getSystemPropertyOrEnvVar("josdk.it.waitForNamespaceDeletion", true);
 
@@ -261,6 +271,11 @@ public T preserveNamespaceOnError(boolean value) {
       return (T) this;
     }
 
+    public T skipNamespaceDeletion(boolean value) {
+      this.skipNamespaceDeletion = value;
+      return (T) this;
+    }
+
     public T waitForNamespaceDeletion(boolean value) {
       this.waitForNamespaceDeletion = value;
       return (T) this;

operator-framework-junit/src/main/java/io/javaoperatorsdk/operator/junit/ClusterDeployedOperatorExtension.java

@@ -51,6 +51,7 @@ private ClusterDeployedOperatorExtension(
       List<HasMetadata> infrastructure,
       Duration infrastructureTimeout,
       boolean preserveNamespaceOnError,
+      boolean skipNamespaceDeletion,
       boolean waitForNamespaceDeletion,
       boolean oneNamespacePerClass,
       KubernetesClient kubernetesClient,
@@ -62,6 +63,7 @@ private ClusterDeployedOperatorExtension(
         infrastructureTimeout,
         oneNamespacePerClass,
         preserveNamespaceOnError,
+        skipNamespaceDeletion,
         waitForNamespaceDeletion,
         kubernetesClient,
         infrastructureKubernetesClient,
@@ -189,6 +191,7 @@ public ClusterDeployedOperatorExtension build() {
           infrastructure,
           infrastructureTimeout,
           preserveNamespaceOnError,
+          skipNamespaceDeletion,
           waitForNamespaceDeletion,
           oneNamespacePerClass,
           kubernetesClient,

operator-framework-junit/src/main/java/io/javaoperatorsdk/operator/junit/LocallyRunOperatorExtension.java

@@ -80,6 +80,7 @@ private LocallyRunOperatorExtension(
       List<CustomResourceDefinition> additionalCustomResourceDefinitionInstances,
       Duration infrastructureTimeout,
       boolean preserveNamespaceOnError,
+      boolean skipNamespaceDeletion,
       boolean waitForNamespaceDeletion,
       boolean oneNamespacePerClass,
       KubernetesClient kubernetesClient,
@@ -94,6 +95,7 @@ private LocallyRunOperatorExtension(
         infrastructureTimeout,
         oneNamespacePerClass,
         preserveNamespaceOnError,
+        skipNamespaceDeletion,
         waitForNamespaceDeletion,
         kubernetesClient,
         infrastructureKubernetesClient,
@@ -541,6 +543,7 @@ public LocallyRunOperatorExtension build() {
           additionalCustomResourceDefinitionInstances,
           infrastructureTimeout,
           preserveNamespaceOnError,
+          skipNamespaceDeletion,
           waitForNamespaceDeletion,
           oneNamespacePerClass,
           kubernetesClient,