Remove MustGather from test-frame

test-frame-log-collector/README.md

@@ -169,43 +169,6 @@ the logs path will then look like this:
 ```
 The tree path will look similarly to above examples, there will be folders for Namespaces matching the specified labels.
 
-### MustGather
-`MustGather` is an annotation which can handle collecting logs which user want automatically in case of test failure or before/after failure.
-It gets configuration passed into `MustGatherController` and call collecting in proper callbacks.
-
-Register `MustGatherController` handlers and configure
-
-```java
-import io.skodjob.testframe.LogCollectorBuilder;
-import io.skodjob.testframe.annotations.MustGather;
-import io.skodjob.testframe.listeners.MustGatherController;
-import io.skodjob.testframe.resources.KubeResourceManager;
-import org.junit.jupiter.api.Test;
-
-@MustGather
-class TestClass() {
-    static {
-        // Setup global log collector and handlers
-        MustGatherController.setupMustGatherController(new LogCollectorBuilder()
-            .withNamespacedResources("sa", "deployment", "configmaps", "secret")
-            .withClusterWideResources("nodes")
-            .withKubeClient(KubeResourceManager.getKubeClient())
-            .withKubeCmdClient(KubeResourceManager.getKubeCmdClient())
-            .withRootFolderPath("/some-path/path/")
-            .build());
-        MustGatherController.setMustGatherCallback(() -> {
-            MustGatherController.getMustGatherController().collectFromNamespaces("test-namespace", "test-namespace-2");
-            MustGatherController.getMustGatherController().collectClusterWideResources();
-        });
-    }
-
-    @Test
-    void test() {
-        ...
-    }
-}
-```
-
 ### Specifying additional folder path
 
 In case that you would like to collect the logs to additional sub-directories of your root folder, the `LogCollector` contains

test-frame-test-examples/src/test/java/io/skodjob/testframe/test/integration/AbstractIT.java

@@ -4,10 +4,6 @@
  */
 package io.skodjob.testframe.test.integration;
 
-import io.skodjob.testframe.LogCollectorBuilder;
-import io.skodjob.testframe.annotations.MustGather;
-import io.skodjob.testframe.listeners.MustGatherController;
-import io.skodjob.testframe.test.integration.helpers.GlobalLogCollectorTestHandler;
 import io.skodjob.testframe.utils.LoggerUtils;
 import io.skodjob.testframe.annotations.ResourceManager;
 import io.skodjob.testframe.annotations.TestVisualSeparator;
@@ -16,17 +12,14 @@
 import io.skodjob.testframe.resources.NamespaceType;
 import io.skodjob.testframe.resources.ServiceAccountType;
 import io.skodjob.testframe.utils.KubeUtils;
-import org.junit.jupiter.api.extension.ExtendWith;
 
 import java.nio.file.Path;
 import java.nio.file.Paths;
 import java.time.LocalDateTime;
 import java.time.format.DateTimeFormatter;
 import java.util.concurrent.atomic.AtomicBoolean;
 
-@ExtendWith(GlobalLogCollectorTestHandler.class) // For testing purpose
 @ResourceManager
-@MustGather
 @TestVisualSeparator
 public abstract class AbstractIT {
     static AtomicBoolean isCreateHandlerCalled = new AtomicBoolean(false);
@@ -58,19 +51,6 @@ public abstract class AbstractIT {
                 LoggerUtils.logResource("Deleted", r);
             }
         });
-
-        // Setup global log collector and handlers
-        MustGatherController.setupMustGatherController(new LogCollectorBuilder()
-            .withNamespacedResources("sa", "deployment", "configmaps", "secret")
-            .withClusterWideResources("nodes")
-            .withKubeClient(KubeResourceManager.getKubeClient())
-            .withKubeCmdClient(KubeResourceManager.getKubeCmdClient())
-            .withRootFolderPath(LOG_DIR.toString())
-            .build());
-        MustGatherController.setMustGatherCallback(() -> {
-            MustGatherController.getMustGatherController().collectFromNamespaces("default");
-            MustGatherController.getMustGatherController().collectClusterWideResources();
-        });
     }
 
     protected String nsName1 = "test";