Remove unnecessary label containing the Prometheus port; Add BREAKING…

… flag to the changelog
stackabletech · Feb 15, 2022 · a4a4da5 · a4a4da5
1 parent c5b2eca
commit a4a4da5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,7 +7,8 @@
 ### Added
 
 - Druid DB connections can now be configured in a custom resource ([#71]).
-- Prometheus metrics enabled ([#128]).
+- BREAKING: Prometheus metrics enabled ([#128]); The `statsdExporterVersion` must be set in the
+  cluster specification.
 
 ### Changed
 

diff --git a/rust/operator-binary/src/superset_controller.rs b/rust/operator-binary/src/superset_controller.rs
@@ -222,7 +222,6 @@ fn build_node_rolegroup_service(
                 statsd_exporter_version(superset).context(NoStatsdExporterVersionSnafu)?,
             )
             .with_label("prometheus.io/scrape", "true")
-            .with_label("prometheus.io/port", METRICS_PORT.to_string())
             .build(),
         spec: Some(ServiceSpec {
             cluster_ip: Some("None".to_string()),