fix: remove pod delete web page dependent test (#2061)

Signed-off-by: Attila Mészáros <[email protected]>

Author: csviri

Diff for: sample-operators/webpage/src/main/java/io/javaoperatorsdk/operator/sample/dependentresource/ConfigMapDependentResource.java

@@ -3,9 +3,6 @@
 import java.util.HashMap;
 import java.util.Map;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
 import io.fabric8.kubernetes.api.model.ConfigMap;
 import io.fabric8.kubernetes.api.model.ConfigMapBuilder;
 import io.fabric8.kubernetes.api.model.ObjectMetaBuilder;
@@ -15,16 +12,13 @@
 import io.javaoperatorsdk.operator.sample.customresource.WebPage;
 
 import static io.javaoperatorsdk.operator.sample.Utils.configMapName;
-import static io.javaoperatorsdk.operator.sample.Utils.deploymentName;
 import static io.javaoperatorsdk.operator.sample.WebPageManagedDependentsReconciler.SELECTOR;
 
 // this annotation only activates when using managed dependents and is not otherwise needed
 @KubernetesDependent(labelSelector = SELECTOR)
 public class ConfigMapDependentResource
     extends CRUDKubernetesDependentResource<ConfigMap, WebPage> {
 
-  private static final Logger log = LoggerFactory.getLogger(ConfigMapDependentResource.class);
-
   public ConfigMapDependentResource() {
     super(ConfigMap.class);
   }
@@ -45,22 +39,4 @@ protected ConfigMap desired(WebPage webPage, Context<WebPage> context) {
         .withData(data)
         .build();
   }
-
-  @Override
-  public ConfigMap update(ConfigMap actual, ConfigMap desired, WebPage primary,
-      Context<WebPage> context) {
-    var res = super.update(actual, desired, primary, context);
-    var ns = actual.getMetadata().getNamespace();
-    log.info("Restarting pods because HTML has changed in {}",
-        ns);
-    // not that this is not necessary, eventually mounted config map would be updated, just this way
-    // is much faster; what is handy for demo purposes.
-    // https://kubernetes.io/docs/tasks/configure-pod-container/configure-pod-configmap/#mounted-configmaps-are-updated-automatically
-    getKubernetesClient()
-        .pods()
-        .inNamespace(ns)
-        .withLabel("app", deploymentName(primary))
-        .delete();
-    return res;
-  }
 }

Diff for: sample-operators/webpage/src/test/java/io/javaoperatorsdk/operator/sample/WebPageOperatorAbstractTest.java

@@ -36,6 +36,7 @@ public abstract class WebPageOperatorAbstractTest {
   public static final String TITLE1 = "Hello Operator World";
   public static final String TITLE2 = "Hello Operator World Title 2";
   public static final int WAIT_SECONDS = 20;
+  public static final int LONG_WAIT_SECONDS = 120;
   public static final Duration POLL_INTERVAL = Duration.ofSeconds(1);
 
   boolean isLocal() {
@@ -68,7 +69,7 @@ void testAddingWebPage() {
     // update part: changing title
     operator().replace(createWebPage(TITLE2));
 
-    await().atMost(Duration.ofSeconds(WAIT_SECONDS))
+    await().atMost(Duration.ofSeconds(LONG_WAIT_SECONDS))
         .pollInterval(POLL_INTERVAL)
         .untilAsserted(() -> {
           String page = httpGetForWebPage(webPage);