test fix

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

Author: csviri

docs/content/en/blog/releases/v5-5-release.md

@@ -11,6 +11,8 @@ more efficient, and exposes a richer, matcher-aware `ResourceOperations` API. Th
 breaking API changes**, but there is one behavioral change around `UpdateControl` — see the
 migration notes.
 
+If you are running on version `5.3.x` or `5.4.0` upgrade is strongly recommend!
+
 ## Key Features
 
 ### Matcher-based updates in `ResourceOperations`

operator-framework/src/test/java/io/javaoperatorsdk/operator/baseapi/latestdistinct/LatestDistinctTestReconciler.java

@@ -20,13 +20,17 @@
 import java.util.Set;
 import java.util.concurrent.atomic.AtomicInteger;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 import io.fabric8.kubernetes.api.model.ConfigMap;
 import io.javaoperatorsdk.operator.api.config.informer.InformerEventSourceConfiguration;
 import io.javaoperatorsdk.operator.api.reconciler.Context;
 import io.javaoperatorsdk.operator.api.reconciler.ControllerConfiguration;
 import io.javaoperatorsdk.operator.api.reconciler.ErrorStatusUpdateControl;
 import io.javaoperatorsdk.operator.api.reconciler.EventSourceContext;
 import io.javaoperatorsdk.operator.api.reconciler.Reconciler;
+import io.javaoperatorsdk.operator.api.reconciler.ResourceOperations;
 import io.javaoperatorsdk.operator.api.reconciler.UpdateControl;
 import io.javaoperatorsdk.operator.processing.event.ResourceID;
 import io.javaoperatorsdk.operator.processing.event.source.EventSource;
@@ -35,6 +39,8 @@
 @ControllerConfiguration
 public class LatestDistinctTestReconciler implements Reconciler<LatestDistinctTestResource> {
 
+  private static final Logger log = LoggerFactory.getLogger(LatestDistinctTestReconciler.class);
+
   public static final String EVENT_SOURCE_1_NAME = "configmap-es-1";
   public static final String EVENT_SOURCE_2_NAME = "configmap-es-2";
   public static final String LABEL_KEY = "configmap-type";
@@ -53,6 +59,7 @@ public UpdateControl<LatestDistinctTestResource> reconcile(
     }
     var allConfigMaps = context.getSecondaryResourcesAsStream(ConfigMap.class).toList();
     if (allConfigMaps.size() < 2) {
+      log.info("Skipping reconciliation, only {} config map in cache", allConfigMaps.size());
       // wait until both informers see the config map
       return UpdateControl.noUpdate();
     }
@@ -66,7 +73,10 @@ public UpdateControl<LatestDistinctTestResource> reconcile(
     resource.getStatus().setConfigMapCount(distinctConfigMaps.size());
     var configMap = distinctConfigMaps.get(0);
     configMap.setData(Map.of(KEY_2, "val2"));
-    var updated = context.resourceOperations().update(configMap);
+    var updated =
+        context
+            .resourceOperations()
+            .update(configMap, ResourceOperations.Options.filterWithOptimisticLocking());
 
     // makes sure that distinct config maps returned
     distinctConfigMaps = context.getSecondaryResourcesAsStream(ConfigMap.class, true).toList();