fix: leader election manager init when not using config overrider (#1636)

Author: nicoloboschi

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

@@ -60,9 +60,9 @@ public Operator(KubernetesClient client, Consumer<ConfigurationServiceOverrider>
   public Operator(KubernetesClient kubernetesClient, ConfigurationService configurationService) {
     this.kubernetesClient =
         kubernetesClient != null ? kubernetesClient : new KubernetesClientBuilder().build();
+    ConfigurationServiceProvider.set(configurationService);
     configurationService.getLeaderElectionConfiguration()
         .ifPresent(c -> leaderElectionManager.init(c, this.kubernetesClient));
-    ConfigurationServiceProvider.set(configurationService);
   }
 
   /** Adds a shutdown hook that automatically calls {@link #stop()} when the app shuts down. */

operator-framework-core/src/test/java/io/javaoperatorsdk/operator/OperatorTest.java

@@ -1,5 +1,7 @@
 package io.javaoperatorsdk.operator;
 
+import java.util.Optional;
+
 import org.junit.jupiter.api.AfterAll;
 import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.BeforeAll;
@@ -12,6 +14,7 @@
 import io.javaoperatorsdk.operator.api.config.AbstractConfigurationService;
 import io.javaoperatorsdk.operator.api.config.ConfigurationServiceProvider;
 import io.javaoperatorsdk.operator.api.config.LeaderElectionConfiguration;
+import io.javaoperatorsdk.operator.api.config.Version;
 import io.javaoperatorsdk.operator.api.reconciler.Context;
 import io.javaoperatorsdk.operator.api.reconciler.ControllerConfiguration;
 import io.javaoperatorsdk.operator.api.reconciler.Reconciler;
@@ -84,6 +87,23 @@ void shouldBeAbleToProvideLeaderElectionConfiguration() {
         .getLeaderElectionConfiguration().orElseThrow().getIdentity().orElseThrow());
   }
 
+  @Test
+  void shouldBeAbleToInitLeaderElectionManagerWithoutOverrider() {
+    ConfigurationServiceProvider.reset();
+    final LeaderElectionConfiguration leaderElectionConfiguration =
+        new LeaderElectionConfiguration("leader-election-test", "namespace", "identity");
+    final AbstractConfigurationService configurationService =
+        new AbstractConfigurationService(Version.UNKNOWN) {
+          @Override
+          public Optional<LeaderElectionConfiguration> getLeaderElectionConfiguration() {
+            return Optional.of(leaderElectionConfiguration);
+          }
+        };
+    new Operator(kubernetesClient, configurationService);
+    assertEquals("identity", ConfigurationServiceProvider.instance()
+        .getLeaderElectionConfiguration().orElseThrow().getIdentity().orElseThrow());
+  }
+
   @ControllerConfiguration
   private static class FooReconciler implements Reconciler<ConfigMap> {