Skip to content

Commit 77afb03

Browse files
cdpreteCosimo Damiano Prete
andauthored
fix(#5079): Enable the KubernetesConverterConfiguration configuration only if also the KubernetesDiscoveryProperties bean is available in the context (#5083)
Co-authored-by: Cosimo Damiano Prete <CosimoDamiano.Prete@six-group.com>
1 parent 6efdfd6 commit 77afb03

2 files changed

Lines changed: 3 additions & 8 deletions

File tree

spring-boot-admin-server-cloud/src/main/java/de/codecentric/boot/admin/server/cloud/config/AdminServerDiscoveryAutoConfiguration.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,7 @@ public EurekaServiceInstanceConverter serviceInstanceConverter() {
8080
}
8181

8282
@Configuration(proxyBeanMethods = false)
83+
@ConditionalOnBean(KubernetesDiscoveryProperties.class)
8384
@ConditionalOnMissingBean({ ServiceInstanceConverter.class })
8485
@ConditionalOnCloudPlatform(CloudPlatform.KUBERNETES)
8586
public static class KubernetesConverterConfiguration {

spring-boot-admin-server-cloud/src/test/java/de/codecentric/boot/admin/server/cloud/config/AdminServerDiscoveryAutoConfigurationTest.java

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919
import com.netflix.discovery.EurekaClient;
2020
import org.junit.jupiter.api.Test;
2121
import org.springframework.boot.autoconfigure.AutoConfigurations;
22-
import org.springframework.boot.context.properties.EnableConfigurationProperties;
2322
import org.springframework.boot.http.client.autoconfigure.reactive.ReactiveHttpClientAutoConfiguration;
2423
import org.springframework.boot.test.context.runner.ApplicationContextRunner;
2524
import org.springframework.boot.webclient.autoconfigure.WebClientAutoConfiguration;
@@ -65,8 +64,8 @@ void eurekaServiceInstanceConverter() {
6564

6665
@Test
6766
void kubernetesServiceInstanceConverter() {
68-
this.contextRunner.withUserConfiguration(KubernetesDiscoveryPropertiesConfiguration.class)
69-
.withBean(DiscoveryClient.class, () -> mock(DiscoveryClient.class))
67+
this.contextRunner.withBean(DiscoveryClient.class, () -> mock(DiscoveryClient.class))
68+
.withBean(KubernetesDiscoveryProperties.class, () -> mock(KubernetesDiscoveryProperties.class))
7069
.withPropertyValues("spring.main.cloud-platform=KUBERNETES")
7170
.run((context) -> assertThat(context.getBean(ServiceInstanceConverter.class))
7271
.isInstanceOf(KubernetesServiceInstanceConverter.class));
@@ -89,9 +88,4 @@ public Registration convert(ServiceInstance instance) {
8988

9089
}
9190

92-
@EnableConfigurationProperties(KubernetesDiscoveryProperties.class)
93-
public static class KubernetesDiscoveryPropertiesConfiguration {
94-
95-
}
96-
9791
}

0 commit comments

Comments
 (0)