fix: actually checking finished status now...

redhat-developer · Mar 16, 2021 · af6f880 · af6f880
1 parent bdfa17c
commit af6f880
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/.../rhoas/src/main/java/com/openshift/cloud/controllers/AbstractCloudServicesController.java b/.../rhoas/src/main/java/com/openshift/cloud/controllers/AbstractCloudServicesController.java
@@ -3,6 +3,7 @@
 import com.openshift.cloud.v1alpha.models.CloudServiceAccountRequest;
 import com.openshift.cloud.v1alpha.models.CloudServicesRequest;
 import com.openshift.cloud.v1alpha.models.KafkaCondition;
+import com.openshift.cloud.v1alpha.models.KafkaCondition.Status;
 import com.openshift.cloud.v1alpha.models.KafkaCondition.Type;
 import com.openshift.cloud.v1alpha.models.KafkaConnection;
 import io.fabric8.kubernetes.client.CustomResource;
@@ -77,7 +78,15 @@ private boolean shouldProcess(T resource) {
             < resource.getMetadata().getGeneration()) {
       return true;
     }
-    return false;
+
+    if (Status.True.equals(finishedCondition.getStatus())) {
+      //everything is up to date and the resource had finished, then skip
+      return false;
+    } else {
+      return true;
+    }
+
+
   }
 
   private List<KafkaCondition> getConditions(T resource) {