add logging to help identify issue with failing e2e test (#2095)

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

Author: csviri

Diff for: operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/event/source/informer/InformerEventSource.java

@@ -123,6 +123,7 @@ public void onAdd(R newResource) {
 
   @Override
   public void onUpdate(R oldObject, R newObject) {
+    log.debug("On updated with old: {} \n new: {}", oldObject, newObject);
     if (log.isDebugEnabled()) {
       log.debug(
           "On update event received for resource id: {} type: {} version: {} old version: {} ",

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

@@ -106,8 +106,9 @@ public UpdateControl<WebPage> reconcile(WebPage webPage, Context<WebPage> contex
           "Creating or updating ConfigMap {} in {}",
           desiredHtmlConfigMap.getMetadata().getName(),
           ns);
-      kubernetesClient.configMaps().inNamespace(ns).resource(desiredHtmlConfigMap)
+      var res = kubernetesClient.configMaps().inNamespace(ns).resource(desiredHtmlConfigMap)
           .createOrReplace();
+      log.debug("Updated config map: {}", res);
     }
 
     var existingDeployment = context.getSecondaryResource(Deployment.class).orElse(null);
@@ -181,10 +182,14 @@ private boolean match(Service desiredService, Service service) {
   }
 
   private boolean match(ConfigMap desiredHtmlConfigMap, ConfigMap existingConfigMap) {
+    log.debug("Actual config map: {}, desired configMap: {}", existingConfigMap,
+        desiredHtmlConfigMap);
     if (existingConfigMap == null) {
       return false;
     } else {
-      return desiredHtmlConfigMap.getData().equals(existingConfigMap.getData());
+      var matched = desiredHtmlConfigMap.getData().equals(existingConfigMap.getData());
+      log.debug("Matched config map: {}", matched);
+      return matched;
     }
   }
 

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

@@ -3,6 +3,9 @@
 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;
@@ -19,12 +22,15 @@
 public class ConfigMapDependentResource
     extends CRUDKubernetesDependentResource<ConfigMap, WebPage> {
 
+  private static final Logger log = LoggerFactory.getLogger(ConfigMapDependentResource.class);
+
   public ConfigMapDependentResource() {
     super(ConfigMap.class);
   }
 
   @Override
   protected ConfigMap desired(WebPage webPage, Context<WebPage> context) {
+    log.debug("Web page spec: {}", webPage.getSpec().getHtml());
     Map<String, String> data = new HashMap<>();
     data.put("index.html", webPage.getSpec().getHtml());
     Map<String, String> labels = new HashMap<>();
@@ -39,4 +45,15 @@ protected ConfigMap desired(WebPage webPage, Context<WebPage> context) {
         .withData(data)
         .build();
   }
+
+  @Override
+  public Result<ConfigMap> match(ConfigMap actualResource, WebPage primary,
+      Context<WebPage> context) {
+    var matched = super.match(actualResource, primary, context);
+    log.debug("Match for config map {} res: {}", actualResource.getMetadata().getName(),
+        matched.matched());
+    return matched;
+  }
+
+
 }