feat: encapsulate Kubernetes dependent configuration

This will help for passing it around as one object instead of passing
the informer configuration part and the owned status separately.

Author: metacosm

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

@@ -0,0 +1,38 @@
+package io.javaoperatorsdk.operator.api.reconciler.dependent;
+
+import java.util.Set;
+
+import io.fabric8.kubernetes.api.model.HasMetadata;
+import io.javaoperatorsdk.operator.api.config.ConfigurationService;
+import io.javaoperatorsdk.operator.processing.event.source.AssociatedSecondaryResourceIdentifier;
+import io.javaoperatorsdk.operator.processing.event.source.PrimaryResourcesRetriever;
+import io.javaoperatorsdk.operator.processing.event.source.informer.InformerConfiguration;
+
+public class KubernetesDependentResourceConfiguration<R extends HasMetadata, P extends HasMetadata>
+    extends InformerConfiguration<R, P> {
+
+  private final boolean owned;
+
+  protected KubernetesDependentResourceConfiguration(
+      ConfigurationService service,
+      String labelSelector, Class<R> resourceClass,
+      PrimaryResourcesRetriever<R> secondaryToPrimaryResourcesIdSet,
+      AssociatedSecondaryResourceIdentifier<P> associatedWith,
+      boolean skipUpdateEventPropagationIfNoChange, Set<String> namespaces, boolean owned) {
+    super(service, labelSelector, resourceClass, secondaryToPrimaryResourcesIdSet, associatedWith,
+        skipUpdateEventPropagationIfNoChange, namespaces);
+    this.owned = owned;
+  }
+
+  public static <R extends HasMetadata, P extends HasMetadata> KubernetesDependentResourceConfiguration<R, P> from(
+      InformerConfiguration<R, P> cfg, boolean owned) {
+    return new KubernetesDependentResourceConfiguration<>(cfg.getConfigurationService(),
+        cfg.getLabelSelector(), cfg.getResourceClass(), cfg.getPrimaryResourcesRetriever(),
+        cfg.getAssociatedResourceIdentifier(), cfg.isSkipUpdateEventPropagationIfNoChange(),
+        cfg.getNamespaces(), owned);
+  }
+
+  public boolean isOwned() {
+    return owned;
+  }
+}

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

@@ -13,14 +13,13 @@
 
 public class KubernetesDependentResourceController<R extends HasMetadata, P extends HasMetadata>
     extends DependentResourceController<R, P> {
-  private final InformerConfiguration<R, P> configuration;
-  private final boolean owned;
+  private final KubernetesDependentResourceConfiguration<R, P> configuration;
   private KubernetesClient client;
   private InformerEventSource<R, P> informer;
 
 
   public KubernetesDependentResourceController(DependentResource<R, P> delegate,
-      InformerConfiguration<R, P> configuration, boolean owned) {
+      KubernetesDependentResourceConfiguration<R, P> configuration) {
     super(delegate);
     // todo: check if we can validate that types actually match properly
     final var associatedPrimaries =
@@ -32,11 +31,12 @@ public KubernetesDependentResourceController(DependentResource<R, P> delegate,
             ? (AssociatedSecondaryResourceIdentifier<P>) delegate
             : configuration.getAssociatedResourceIdentifier();
 
-    this.configuration = InformerConfiguration.from(configuration)
+    final var augmented = InformerConfiguration.from(configuration)
         .withPrimaryResourcesRetriever(associatedPrimaries)
         .withAssociatedSecondaryResourceIdentifier(associatedSecondary)
         .build();
-    this.owned = owned;
+    this.configuration =
+        KubernetesDependentResourceConfiguration.from(augmented, configuration.isOwned());
   }
 
   @Override
@@ -69,6 +69,6 @@ public R getFor(P primary, Context context) {
   }
 
   public boolean owned() {
-    return owned;
+    return configuration.isOwned();
   }
 }

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/event/source/informer/InformerConfiguration.java

@@ -19,7 +19,7 @@ public class InformerConfiguration<R extends HasMetadata, P extends HasMetadata>
   private final AssociatedSecondaryResourceIdentifier<P> associatedWith;
   private final boolean skipUpdateEventPropagationIfNoChange;
 
-  private InformerConfiguration(ConfigurationService service, String labelSelector,
+  protected InformerConfiguration(ConfigurationService service, String labelSelector,
       Class<R> resourceClass,
       PrimaryResourcesRetriever<R> secondaryToPrimaryResourcesIdSet,
       AssociatedSecondaryResourceIdentifier<P> associatedWith,

operator-framework/src/main/java/io/javaoperatorsdk/operator/config/runtime/AnnotationConfiguration.java

@@ -15,6 +15,7 @@
 import io.javaoperatorsdk.operator.api.config.KubernetesDependent;
 import io.javaoperatorsdk.operator.api.reconciler.ControllerConfiguration;
 import io.javaoperatorsdk.operator.api.reconciler.Reconciler;
+import io.javaoperatorsdk.operator.api.reconciler.dependent.KubernetesDependentResourceConfiguration;
 import io.javaoperatorsdk.operator.api.reconciler.dependent.KubernetesDependentResourceController;
 import io.javaoperatorsdk.operator.processing.event.source.controller.ResourceEventFilter;
 import io.javaoperatorsdk.operator.processing.event.source.controller.ResourceEventFilters;
@@ -153,7 +154,8 @@ public List<DependentResource> getDependentResources() {
                 .skippingEventPropagationIfUnchanged(skipIfUnchanged)
                 .withNamespaces(namespaces)
                 .build();
-            dependent = new KubernetesDependentResourceController(dependent, configuration, owned);
+            dependent = new KubernetesDependentResourceController(dependent,
+                KubernetesDependentResourceConfiguration.from(configuration, owned));
           }
 
           dependents.add(dependent);