Skip to content

Commit ddc45ea

Browse files
committed
Merge pull request #44069 from nosan
* pr/44069: Polish PrometheusPushGatewayConfiguration Closes gh-44069
2 parents 2b72e11 + 1749235 commit ddc45ea

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,9 +37,9 @@
3737
import org.springframework.boot.autoconfigure.AutoConfiguration;
3838
import org.springframework.boot.autoconfigure.EnableAutoConfiguration;
3939
import org.springframework.boot.autoconfigure.condition.ConditionalOnBean;
40+
import org.springframework.boot.autoconfigure.condition.ConditionalOnBooleanProperty;
4041
import org.springframework.boot.autoconfigure.condition.ConditionalOnClass;
4142
import org.springframework.boot.autoconfigure.condition.ConditionalOnMissingBean;
42-
import org.springframework.boot.autoconfigure.condition.ConditionalOnProperty;
4343
import org.springframework.boot.context.properties.EnableConfigurationProperties;
4444
import org.springframework.boot.context.properties.source.MutuallyExclusiveConfigurationPropertiesException;
4545
import org.springframework.context.annotation.Bean;
@@ -102,7 +102,7 @@ PrometheusScrapeEndpoint prometheusEndpoint(PrometheusRegistry prometheusRegistr
102102
*/
103103
@Configuration(proxyBeanMethods = false)
104104
@ConditionalOnClass(PushGateway.class)
105-
@ConditionalOnProperty(prefix = "management.prometheus.metrics.export.pushgateway", name = "enabled")
105+
@ConditionalOnBooleanProperty("management.prometheus.metrics.export.pushgateway.enabled")
106106
static class PrometheusPushGatewayConfiguration {
107107

108108
/**

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

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -186,6 +186,14 @@ void withPushGatewayEnabled(CapturedOutput output) {
186186
});
187187
}
188188

189+
@Test
190+
void withPushGatewayDisabled() {
191+
this.contextRunner.withConfiguration(AutoConfigurations.of(ManagementContextAutoConfiguration.class))
192+
.withPropertyValues("management.prometheus.metrics.export.pushgateway.enabled=false")
193+
.withUserConfiguration(BaseConfiguration.class)
194+
.run((context) -> assertThat(context).doesNotHaveBean(PrometheusPushGatewayManager.class));
195+
}
196+
189197
@Test
190198
void withPushGatewayNoBasicAuth() {
191199
this.contextRunner.withConfiguration(AutoConfigurations.of(ManagementContextAutoConfiguration.class))

0 commit comments

Comments
 (0)