Skip to content

Commit da759f2

Browse files
Jon Schneidersnicoll
authored andcommitted
Restore behavior of management.metrics.export.simple.enabled
See gh-12106
1 parent 19ce68d commit da759f2

File tree

4 files changed

+13
-6
lines changed

4 files changed

+13
-6
lines changed

spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/metrics/export/simple/SimpleMetricsExportAutoConfiguration.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
import org.springframework.boot.autoconfigure.EnableAutoConfiguration;
2929
import org.springframework.boot.autoconfigure.condition.ConditionalOnBean;
3030
import org.springframework.boot.autoconfigure.condition.ConditionalOnMissingBean;
31+
import org.springframework.boot.autoconfigure.condition.ConditionalOnProperty;
3132
import org.springframework.boot.context.properties.EnableConfigurationProperties;
3233
import org.springframework.context.annotation.Bean;
3334
import org.springframework.context.annotation.Configuration;
@@ -46,6 +47,7 @@
4647
@ConditionalOnBean(Clock.class)
4748
@EnableConfigurationProperties(SimpleProperties.class)
4849
@ConditionalOnMissingBean(MeterRegistry.class)
50+
@ConditionalOnProperty(prefix = "management.metrics.export.simple", name = "enabled", havingValue = "true", matchIfMissing = true)
4951
public class SimpleMetricsExportAutoConfiguration {
5052

5153
@Bean

spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/metrics/export/simple/SimpleProperties.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,8 @@
3434
public class SimpleProperties {
3535

3636
/**
37-
* Enable publishing to the backend.
37+
* Enable in-memory metrics that aren't published anywhere (allows you to see
38+
* what metrics are collected in the metrics actuator endpoint).
3839
*/
3940
private boolean enabled;
4041

spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/metrics/export/simple/SimplePropertiesConfigAdapter.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -41,11 +41,6 @@ public String get(String k) {
4141
return null;
4242
}
4343

44-
@Override
45-
public boolean enabled() {
46-
return get(SimpleProperties::getEnabled, SimpleConfig.super::enabled);
47-
}
48-
4944
@Override
5045
public Duration step() {
5146
return get(SimpleProperties::getStep, SimpleConfig.super::step);

spring-boot-project/spring-boot-actuator-autoconfigure/src/test/java/org/springframework/boot/actuate/autoconfigure/metrics/export/simple/SimpleMetricsExportAutoConfigurationTests.java

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,15 @@ public void autoConfiguresConfigAndMeterRegistry() {
5151
.hasSingleBean(Clock.class).hasSingleBean(SimpleConfig.class));
5252
}
5353

54+
@Test
55+
public void backsOffWhenSpecificallyDisabled() {
56+
this.contextRunner.withUserConfiguration(BaseConfiguration.class)
57+
.withPropertyValues("management.metrics.export.simple.enabled=false")
58+
.run((context) -> assertThat(context)
59+
.doesNotHaveBean(SimpleMeterRegistry.class)
60+
.doesNotHaveBean(SimpleConfig.class));
61+
}
62+
5463
@Test
5564
public void allowsConfigToBeCustomized() {
5665
this.contextRunner.withUserConfiguration(CustomConfigConfiguration.class)

0 commit comments

Comments
 (0)