zuul-scheduler: zuul-statsd sidecar use default container Limit profile

The container used to be OOM killed due to memory limit. Also bump statsd-exporter bump version to 0.27.1. Change-Id: Id99e9b6d4c25244bfcd9ebd3a8978ae15b09ef87
softwarefactory-project · Sep 10, 2024 · f3d16c0 · f3d16c0
1 parent c4293ec
commit f3d16c0
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/controllers/libs/base/static/images.yaml b/controllers/libs/base/static/images.yaml
@@ -58,7 +58,7 @@ images:
       version: v1.8.2
     - name: statsd-exporter
       container: quay.io/prometheus/statsd-exporter
-      version: v0.26.1
+      version: v0.27.1
     - name: fluentbit
       container: cr.fluentbit.io/fluent/fluent-bit
       version: "2.1.10"

diff --git a/controllers/libs/monitoring/monitoring.go b/controllers/libs/monitoring/monitoring.go
@@ -187,7 +187,6 @@ func MkStatsdExporterSideCarContainer(serviceName string, configVolumeName strin
 	sidecar.Args = args
 	sidecar.VolumeMounts = volumeMounts
 	sidecar.Ports = ports
-	base.SetContainerLimitsLowProfile(&sidecar)
 	return sidecar
 }
 

diff --git a/doc/reference/CHANGELOG.md b/doc/reference/CHANGELOG.md
@@ -10,6 +10,10 @@ All notable changes to this project will be documented in this file.
 
 ### Removed
 ### Changed
+
+- zuul-scheduler: zuul-statsd sidecar use default container Limit profile
+- zuul-scheduler: statds-exporter bump version to 0.27.1
+
 ### Fixed
 
 - weeder: ensure the service is restarted after backup restore to load the new ZK certs.