fix: minor improvements on logging and tests related to temp cache (#1115)

Author: csviri

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/event/source/controller/TemporaryResourceCache.java

@@ -88,6 +88,8 @@ public Optional<R> get(ResourceID resourceID) {
       log.debug("Resource found in temporal cache for Resource ID: {}", resourceID);
       return resource;
     } else {
+      log.debug("Resource not found in temporal cache reading it from informer cache," +
+          " for Resource ID: {}", resourceID);
       return super.get(resourceID);
     }
   }

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/event/source/informer/ManagedInformerEventSource.java

@@ -73,6 +73,7 @@ public synchronized void putUpdatedResource(T newResource, String previousResour
       cache.put(resourceId, newResource);
     } else {
       // if something is in cache it's surely obsolete now
+      log.debug("Trying to remove an obsolete resource from cache for id: {}", resourceId);
       cache.remove(resourceId);
     }
   }

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/event/source/informer/TemporaryResourceCache.java

@@ -28,7 +28,7 @@ public class StandaloneDependentTestReconciler
     KubernetesClientAware, ErrorStatusHandler<StandaloneDependentTestCustomResource> {
 
   private KubernetesClient kubernetesClient;
-  private boolean errorOccurred = false;
+  private volatile boolean errorOccurred = false;
 
   DeploymentDependentResource deploymentDependent;