kayenta icon indicating copy to clipboard operation
kayenta copied to clipboard

Enable dynamic port allocation for Prometheus and management server during integration tests

Open rahul-chekuri opened this issue 10 months ago • 1 comments

  • Renamed EmbeddedPrometheusBootstrapConfiguration as TestPrometheusConfiguration and now TestPrometheusConfiguration dynamically allocate and register Prometheus container ports without using hardcoded ports.

  • Removed dependency on predefined server.port and management.server.port values for integration tests.

  • Management server port is dynamically obtained during runtime and exposed for Prometheus to scrape metrics.

  • Prometheus container’s mapped port is dynamically registered into the environment (embedded.prometheus.port) at runtime.

  • Fixed ManagementTest to use the dynamically registered Prometheus port (embedded.prometheus.port).

  • Added logic in BaseIntegrationTest to update dynamicEndpoint to PrometheusRemoteService

rahul-chekuri avatar Apr 21 '25 10:04 rahul-chekuri

The following commits need their title changed:

  • 2916175910a82450f02bfcb66198f09d1432f7c3: Replace hardcoded ports with random ports logic.

  • 9457ba86d382cc673564cc4ae1f1a29364ce6642: Replace hardcoded ports with random ports logic.

Please format your commit title into the form:

<type>(<scope>): <subject>, e.g. fix(kubernetes): address NPE in status check

This allows us to easily generate changelogs & determine semantic version numbers when cutting releases. You can read more about commit conventions here.

spinnakerbot avatar Apr 21 '25 10:04 spinnakerbot