fix: Reorder setting visited flag and readyPostcondition result (#2886)

Signed-off-by: Adrian Kłoskowski <[email protected]>

Author: ds-akloskowski

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/dependent/workflow/WorkflowReconcileExecutor.java

@@ -144,9 +144,11 @@ protected void doRun(DependentResourceNode<R, P> dependentResourceNode) {
       log.debug("Reconciling for primary: {} node: {} ", primaryID, dependentResourceNode);
       ReconcileResult reconcileResult = dependentResource.reconcile(primary, context);
       final var detailBuilder = createOrGetResultFor(dependentResourceNode);
-      detailBuilder.withReconcileResult(reconcileResult).markAsVisited();
 
-      if (isConditionMet(dependentResourceNode.getReadyPostcondition(), dependentResourceNode)) {
+      boolean isReadyPostconditionMet =
+          isConditionMet(dependentResourceNode.getReadyPostcondition(), dependentResourceNode);
+      detailBuilder.withReconcileResult(reconcileResult).markAsVisited();
+      if (isReadyPostconditionMet) {
         log.debug(
             "Setting already reconciled for: {} primaryID: {}", dependentResourceNode, primaryID);
         handleDependentsReconcile(dependentResourceNode);