improve: DependentResourceCrossRefIT cleanup and logging (#2883)

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

Author: csviri

operator-framework/src/test/java/io/javaoperatorsdk/operator/dependent/dependentresourcecrossref/DependentResourceCrossRefIT.java

@@ -44,6 +44,22 @@ void dependentResourceCanReferenceEachOther() {
                 assertThat(operator.get(Secret.class, TEST_RESOURCE_NAME + i)).isNotNull();
               }
             });
+
+    for (int i = 0; i < EXECUTION_NUMBER; i++) {
+      operator.delete(testResource(i));
+    }
+    await()
+        .timeout(Duration.ofSeconds(30))
+        .untilAsserted(
+            () -> {
+              for (int i = 0; i < EXECUTION_NUMBER; i++) {
+                assertThat(
+                        operator.get(
+                            DependentResourceCrossRefResource.class,
+                            testResource(i).getMetadata().getName()))
+                    .isNull();
+              }
+            });
   }
 
   DependentResourceCrossRefResource testResource(int n) {

operator-framework/src/test/java/io/javaoperatorsdk/operator/dependent/dependentresourcecrossref/DependentResourceCrossRefReconciler.java

@@ -5,6 +5,9 @@
 import java.util.Map;
 import java.util.concurrent.atomic.AtomicInteger;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 import io.fabric8.kubernetes.api.model.ConfigMap;
 import io.fabric8.kubernetes.api.model.ObjectMetaBuilder;
 import io.fabric8.kubernetes.api.model.Secret;
@@ -26,6 +29,8 @@
 @ControllerConfiguration
 public class DependentResourceCrossRefReconciler
     implements Reconciler<DependentResourceCrossRefResource> {
+  private static final Logger log =
+      LoggerFactory.getLogger(DependentResourceCrossRefReconciler.class);
 
   public static final String SECRET_NAME = "secret";
   private final AtomicInteger numberOfExecutions = new AtomicInteger(0);
@@ -48,6 +53,7 @@ public ErrorStatusUpdateControl<DependentResourceCrossRefResource> updateErrorSt
       DependentResourceCrossRefResource resource,
       Context<DependentResourceCrossRefResource> context,
       Exception e) {
+    log.error("Status update on error", e);
     errorHappened = true;
     return ErrorStatusUpdateControl.noStatusUpdate();
   }