From 6ae6a3b2bd91c6b25d77c464121f0b54a3a67831 Mon Sep 17 00:00:00 2001 From: Haytham Mohamed Date: Tue, 30 Jul 2019 16:34:34 -0500 Subject: [PATCH] restart endpoint does not need to be enabled when refresh is used by default fixes gh-402 --- .../config/reload/ConfigReloadAutoConfiguration.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/spring-cloud-kubernetes-config/src/main/java/org/springframework/cloud/kubernetes/config/reload/ConfigReloadAutoConfiguration.java b/spring-cloud-kubernetes-config/src/main/java/org/springframework/cloud/kubernetes/config/reload/ConfigReloadAutoConfiguration.java index b811f827..880319b6 100644 --- a/spring-cloud-kubernetes-config/src/main/java/org/springframework/cloud/kubernetes/config/reload/ConfigReloadAutoConfiguration.java +++ b/spring-cloud-kubernetes-config/src/main/java/org/springframework/cloud/kubernetes/config/reload/ConfigReloadAutoConfiguration.java @@ -38,6 +38,7 @@ import org.springframework.context.annotation.Configuration; import org.springframework.core.env.AbstractEnvironment; import org.springframework.scheduling.annotation.EnableAsync; import org.springframework.scheduling.annotation.EnableScheduling; +import org.springframework.util.Assert; /** * Definition of beans needed for the automatic reload of configuration. @@ -110,9 +111,11 @@ public class ConfigReloadAutoConfiguration { @ConditionalOnMissingBean public ConfigurationUpdateStrategy configurationUpdateStrategy( ConfigReloadProperties properties, ConfigurableApplicationContext ctx, - RestartEndpoint restarter, ContextRefresher refresher) { + @Autowired(required = false) RestartEndpoint restarter, + ContextRefresher refresher) { switch (properties.getStrategy()) { case RESTART_CONTEXT: + Assert.notNull(restarter, "Restart endpoint is not enabled"); return new ConfigurationUpdateStrategy(properties.getStrategy().name(), restarter::restart); case REFRESH: