fix: javadoc problems preventing snapshot release (#1848)

Author: csviri

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/Operator.java

@@ -43,6 +43,7 @@ public Operator(KubernetesClient kubernetesClient) {
   }
 
   /**
+   * @param configurationService implementation
    * @deprecated Use {@link #Operator(Consumer)} instead
    */
   @Deprecated(forRemoval = true)
@@ -173,6 +174,7 @@ public void stop() throws OperatorException {
    *
    * @param reconciler the reconciler to register
    * @param <P> the {@code CustomResource} type associated with the reconciler
+   * @return registered controller
    * @throws OperatorException if a problem occurred during the registration process
    */
   public <P extends HasMetadata> RegisteredController<P> register(Reconciler<P> reconciler)
@@ -192,6 +194,7 @@ public <P extends HasMetadata> RegisteredController<P> register(Reconciler<P> re
    * @param reconciler part of the reconciler to register
    * @param configuration the configuration with which we want to register the reconciler
    * @param <P> the {@code HasMetadata} type associated with the reconciler
+   * @return registered controller
    * @throws OperatorException if a problem occurred during the registration process
    */
   public <P extends HasMetadata> RegisteredController<P> register(Reconciler<P> reconciler,
@@ -230,6 +233,7 @@ public <P extends HasMetadata> RegisteredController<P> register(Reconciler<P> re
    *
    * @param reconciler part of the reconciler to register
    * @param configOverrider consumer to use to change config values
+   * @return registered controller
    * @param <P> the {@code HasMetadata} type associated with the reconciler
    */
   public <P extends HasMetadata> RegisteredController<P> register(Reconciler<P> reconciler,

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/config/ConfigurationService.java

@@ -199,6 +199,8 @@ default Optional<LeaderElectionConfiguration> getLeaderElectionConfiguration() {
    * if false, the startup will ignore recoverable errors, caused for example by RBAC issues, and
    * will try to reconnect periodically in the background.
    * </p>
+   *
+   * @return actual value described above
    */
   default boolean stopOnInformerErrorDuringStartup() {
     return true;
@@ -208,6 +210,8 @@ default boolean stopOnInformerErrorDuringStartup() {
    * Timeout for cache sync. In other words source start timeout. Note that is
    * "stopOnInformerErrorDuringStartup" is true the operator will stop on timeout. Default is 2
    * minutes.
+   *
+   * @return Duration of sync timeout
    */
   default Duration cacheSyncTimeout() {
     return Duration.ofMinutes(2);
@@ -216,6 +220,8 @@ default Duration cacheSyncTimeout() {
   /**
    * Handler for an informer stop. Informer stops if there is a non-recoverable error. Like received
    * a resource that cannot be deserialized.
+   *
+   * @return an optional InformerStopHandler
    */
   default Optional<InformerStoppedHandler> getInformerStoppedHandler() {
     return Optional.of((informer, ex) -> {

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/config/ConfigurationServiceOverrider.java

@@ -238,6 +238,8 @@ public ResourceClassResolver getResourceClassResolver() {
 
   /**
    * @deprecated Use {@link ConfigurationServiceProvider#overrideCurrent(Consumer)} instead
+   * @param original that will be overriding
+   * @return current overrider
    */
   @Deprecated(since = "2.2.0")
   public static ConfigurationServiceOverrider override(ConfigurationService original) {

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/config/ControllerConfigurationOverrider.java

@@ -106,6 +106,8 @@ public ControllerConfigurationOverrider<R> watchingAllNamespaces() {
   }
 
   /**
+   * @param retry configuration
+   * @return current instance of overrider
    * @deprecated Use {@link #withRetry(Retry)} instead
    */
   @Deprecated(forRemoval = true)

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/config/ResolvedControllerConfiguration.java

@@ -156,6 +156,8 @@ public ResourceEventFilter<P> getEventFilter() {
 
   /**
    * @deprecated Use {@link OnAddFilter}, {@link OnUpdateFilter} and {@link GenericFilter} instead
+   *
+   * @param eventFilter
    */
   @Deprecated(forRemoval = true)
   protected void setEventFilter(ResourceEventFilter<P> eventFilter) {

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/config/informer/InformerConfiguration.java

@@ -78,6 +78,8 @@ public <P extends HasMetadata> PrimaryToSecondaryMapper<P> getPrimaryToSecondary
    * {@link io.javaoperatorsdk.operator.RegisteredController}). If true, changing the target
    * namespaces of a controller would result to change target namespaces for the
    * InformerEventSource.
+   *
+   * @return if namespace changes should be followed
    */
   boolean followControllerNamespaceChanges();
 

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/monitoring/Metrics.java

@@ -23,6 +23,8 @@ public interface Metrics {
 
   /**
    * Do initialization if necessary;
+   *
+   * @param controller callback
    */
   default void controllerRegistered(Controller<? extends HasMetadata> controller) {}
 
@@ -36,6 +38,9 @@ default void controllerRegistered(Controller<? extends HasMetadata> controller)
   default void receivedEvent(Event event, Map<String, Object> metadata) {}
 
   /**
+   * @param metadata additional metadata
+   * @param resourceID of primary resource
+   * @param retryInfo for current execution
    * @deprecated Use {@link #reconcileCustomResource(HasMetadata, RetryInfo, Map)} instead
    */
   @Deprecated(forRemoval = true)
@@ -56,6 +61,9 @@ default void reconcileCustomResource(HasMetadata resource, RetryInfo retryInfo,
   }
 
   /**
+   * @param exception actual exception
+   * @param metadata additional metadata
+   * @param resourceID of primary resource
    * @deprecated Use {@link #failedReconciliation(HasMetadata, Exception, Map)} instead
    */
   @Deprecated(forRemoval = true)
@@ -84,7 +92,7 @@ default void reconciliationExecutionFinished(HasMetadata resource,
       Map<String, Object> metadata) {}
 
   /**
-   *
+   * @param resourceID of primary resource
    * @deprecated Use (and implement) {@link #cleanupDoneFor(ResourceID, Map)} instead
    */
   @Deprecated
@@ -102,7 +110,7 @@ default void cleanupDoneFor(ResourceID resourceID) {
   default void cleanupDoneFor(ResourceID resourceID, Map<String, Object> metadata) {}
 
   /**
-   *
+   * @param resourceID of primary resource
    * @deprecated Use (and implement) {@link #finishedReconciliation(ResourceID, Map)} instead
    */
   @Deprecated
@@ -111,6 +119,8 @@ default void finishedReconciliation(ResourceID resourceID) {
   }
 
   /**
+   * @param resourceID of primary resource
+   * @param metadata additional metadata
    * @deprecated Use {@link #finishedReconciliation(HasMetadata, Map)} instead
    */
   @Deprecated(forRemoval = true)

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

@@ -75,14 +75,22 @@
 
   /**
    * Filter of onAdd events of resources.
+   *
+   * @return on-add filter
    **/
   Class<? extends OnAddFilter> onAddFilter() default OnAddFilter.class;
 
-  /** Filter of onUpdate events of resources. */
+  /**
+   * Filter of onUpdate events of resources.
+   *
+   * @return on-update filter
+   */
   Class<? extends OnUpdateFilter> onUpdateFilter() default OnUpdateFilter.class;
 
   /**
    * Filter applied to all operations (add, update, delete). Used to ignore some resources.
+   *
+   * @return generic filter
    **/
   Class<? extends GenericFilter> genericFilter() default GenericFilter.class;
 

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

@@ -8,6 +8,7 @@ public interface Reconciler<R extends HasMetadata> {
    * The implementation of this operation is required to be idempotent. Always use the UpdateControl
    * object to make updates on custom resource if possible.
    *
+   * @throws Exception from the custom implementation
    * @param resource the resource that has been created or updated
    * @param context the context with which the operation is executed
    * @return UpdateControl to manage updates on the custom resource (usually the status) after

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

@@ -10,6 +10,8 @@ default void useEventSourceWithName(String name) {}
   /**
    * Throws {@link EventSourceNotFoundException} an exception if the target event source to use is
    * not found.
+   *
+   * @param eventSourceRetriever for event sources
    */
   void resolveEventSource(EventSourceRetriever<P> eventSourceRetriever)
       throws EventSourceNotFoundException;