Skip to content

ZOOKEEPER-4972: Fix flaky tests in PrometheusMetricsProviderConfigTest - #2311

Merged
anmolnar merged 1 commit into
apache:masterfrom
kezhuw:ZOOKEEPER-4972-fix-flaky-PrometheusMetricsProviderConfigTest-tests
Sep 19, 2025
Merged

anmolnar merged 1 commit into
apache:masterfrom
kezhuw:ZOOKEEPER-4972-fix-flaky-PrometheusMetricsProviderConfigTest-tests

Conversation

@kezhuw

@kezhuw kezhuw commented Aug 26, 2025

Copy link
Copy Markdown
Member

Those tests use hard written ports to listen.

org.apache.zookeeper.metrics.MetricsProviderLifeCycleException: java.io.IOException: Failed to bind to /0.0.0.0:50512
	at org.apache.zookeeper.metrics.prometheus.PrometheusMetricsProvider.start(PrometheusMetricsProvider.java:213)
	at org.apache.zookeeper.metrics.prometheus.PrometheusMetricsProviderConfigTest.testValidHttpsAndHttpConfig(PrometheusMetricsProviderConfigTest.java:88)

Those tests use hard written ports to listen.

```
org.apache.zookeeper.metrics.MetricsProviderLifeCycleException: java.io.IOException: Failed to bind to /0.0.0.0:50512
	at org.apache.zookeeper.metrics.prometheus.PrometheusMetricsProvider.start(PrometheusMetricsProvider.java:213)
	at org.apache.zookeeper.metrics.prometheus.PrometheusMetricsProviderConfigTest.testValidHttpsAndHttpConfig(PrometheusMetricsProviderConfigTest.java:88)
```
@anmolnar
anmolnar merged commit d8e5217 into apache:master Sep 19, 2025
16 checks passed
@anmolnar

Copy link
Copy Markdown
Contributor

Merged to master branch only. Thanks @kezhuw !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants