naming

Signed-off-by: Attila Mészáros <[email protected]>

Author: csviri

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/reconciler/Context.java

@@ -73,7 +73,7 @@ default <R> Stream<R> getSecondaryResourcesAsStream(Class<R> expectedType) {
    */
   boolean isNextReconciliationImminent();
 
-  boolean isDeleteEventPresent();
+  boolean isPrimaryResourceDeleted();
 
-  boolean isDeleteFinalStateUnknown();
+  boolean isPrimaryResourceFinalStateUnknown();
 }

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/reconciler/DefaultContext.java

@@ -24,21 +24,21 @@ public class DefaultContext<P extends HasMetadata> implements Context<P> {
   private final ControllerConfiguration<P> controllerConfiguration;
   private final DefaultManagedWorkflowAndDependentResourceContext<P>
       defaultManagedDependentResourceContext;
-  private final boolean isDeleteEventPresent;
-  private final boolean isDeleteFinalStateUnknown;
+  private final boolean primaryResourceDeleted;
+  private final boolean primaryResourceFinalStateUnknown;
 
   public DefaultContext(
       RetryInfo retryInfo,
       Controller<P> controller,
       P primaryResource,
-      boolean isDeleteEventPresent,
-      boolean isDeleteFinalStateUnknown) {
+      boolean primaryResourceDeleted,
+      boolean primaryResourceFinalStateUnknown) {
     this.retryInfo = retryInfo;
     this.controller = controller;
     this.primaryResource = primaryResource;
     this.controllerConfiguration = controller.getConfiguration();
-    this.isDeleteEventPresent = isDeleteEventPresent;
-    this.isDeleteFinalStateUnknown = isDeleteFinalStateUnknown;
+    this.primaryResourceDeleted = primaryResourceDeleted;
+    this.primaryResourceFinalStateUnknown = primaryResourceFinalStateUnknown;
     this.defaultManagedDependentResourceContext =
         new DefaultManagedWorkflowAndDependentResourceContext<>(controller, primaryResource, this);
   }
@@ -129,13 +129,13 @@ public boolean isNextReconciliationImminent() {
   }
 
   @Override
-  public boolean isDeleteEventPresent() {
-    return isDeleteEventPresent;
+  public boolean isPrimaryResourceDeleted() {
+    return primaryResourceDeleted;
   }
 
   @Override
-  public boolean isDeleteFinalStateUnknown() {
-    return isDeleteFinalStateUnknown;
+  public boolean isPrimaryResourceFinalStateUnknown() {
+    return primaryResourceFinalStateUnknown;
   }
 
   public DefaultContext<P> setRetryInfo(RetryInfo retryInfo) {

operator-framework/src/test/java/io/javaoperatorsdk/operator/baseapi/propagateallevent/onlyreconcile/PropagateEventReconciler.java

@@ -71,22 +71,22 @@ public UpdateControl<PropagateAllEventCustomResource> reconcile(
       throw new RuntimeException("On purpose exception for missing annotation");
     }
 
-    if (primary.isMarkedForDeletion() && !context.isDeleteEventPresent()) {
+    if (primary.isMarkedForDeletion() && !context.isPrimaryResourceDeleted()) {
       setEventOnMarkedForDeletion(true);
       if (getUseFinalizer() && primary.hasFinalizer(FINALIZER)) {
         log.info("Removing finalizer");
         FinalizerUtils.removeFinalizer(primary, FINALIZER, context);
       }
     }
 
-    if (context.isDeleteEventPresent()
+    if (context.isPrimaryResourceDeleted()
         && isFirstDeleteEvent()
         && isThrowExceptionOnFirstDeleteEvent()) {
       isFirstDeleteEvent = false;
       throw new RuntimeException("On purpose exception");
     }
 
-    if (context.isDeleteEventPresent()) {
+    if (context.isPrimaryResourceDeleted()) {
       setDeleteEventPresent(true);
     }
     log.info("Reconciliation finished");