From f3d16c00b3f83f4dee138fc6b8973c53ee766bfd Mon Sep 17 00:00:00 2001 From: Fabien Boucher Date: Tue, 10 Sep 2024 16:44:20 +0200 Subject: [PATCH] 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 --- controllers/libs/base/static/images.yaml | 2 +- controllers/libs/monitoring/monitoring.go | 1 - doc/reference/CHANGELOG.md | 4 ++++ 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/controllers/libs/base/static/images.yaml b/controllers/libs/base/static/images.yaml index 9ecc4ce3..a363c341 100644 --- 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 index bd874593..3773cbcd 100644 --- 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 index 5c543713..08e2ad06 100644 --- 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.