From 85e95ad7ae44dd730d39630b45d67f78bfd12ffd Mon Sep 17 00:00:00 2001 From: Brendan Kellam Date: Wed, 2 Sep 2026 18:13:52 -0700 Subject: [PATCH 1/2] fix: use legacy metrics exporter images --- charts/sourcebot/README.md | 2 ++ .../tests/postgresql_metrics_image_test.yaml | 14 ++++++++++++++ .../tests/redis_metrics_image_test.yaml | 18 ++++++++++++++++++ charts/sourcebot/values.yaml | 10 ++++++++++ 4 files changed, 44 insertions(+) create mode 100644 charts/sourcebot/tests/postgresql_metrics_image_test.yaml create mode 100644 charts/sourcebot/tests/redis_metrics_image_test.yaml diff --git a/charts/sourcebot/README.md b/charts/sourcebot/README.md index e806954..fb3323e 100644 --- a/charts/sourcebot/README.md +++ b/charts/sourcebot/README.md @@ -37,6 +37,7 @@ Sourcebot is a self-hosted tool that helps you understand your codebase. | postgresql.deploy | bool | `true` | Deploy PostgreSQL subchart. Set to false to use an external PostgreSQL instance. | | postgresql.host | string | `""` | PostgreSQL host (only used if deploy is false) | | postgresql.image.repository | string | `"bitnamilegacy/postgresql"` | Overwrite default repository of helm chart to point to non-paid bitnami images | +| postgresql.metrics.image.repository | string | `"bitnamilegacy/postgres-exporter"` | Overwrite default metrics exporter repository to point to non-paid bitnami images | | postgresql.port | int | `5432` | PostgreSQL port | | postgresql.primary.persistence.enabled | bool | `true` | | | postgresql.primary.persistence.size | string | `"8Gi"` | | @@ -52,6 +53,7 @@ Sourcebot is a self-hosted tool that helps you understand your codebase. | redis.deploy | bool | `true` | Deploy Redis/Valkey subchart. Set to false to use an external Redis instance. | | redis.host | string | `""` | Redis host (only used if deploy is false) | | redis.image.repository | string | `"bitnamilegacy/valkey"` | Overwrite default repository of helm chart to point to non-paid bitnami images | +| redis.metrics.image.repository | string | `"bitnamilegacy/redis-exporter"` | Overwrite default metrics exporter repository to point to non-paid bitnami images | | redis.port | int | `6379` | Redis port | | redis.primary.resources.limits.cpu | string | `"1"` | CPU limit for the Redis container | | redis.primary.resources.limits.memory | string | `"1.5Gi"` | Memory limit for the Redis container | diff --git a/charts/sourcebot/tests/postgresql_metrics_image_test.yaml b/charts/sourcebot/tests/postgresql_metrics_image_test.yaml new file mode 100644 index 0000000..86673c2 --- /dev/null +++ b/charts/sourcebot/tests/postgresql_metrics_image_test.yaml @@ -0,0 +1,14 @@ +suite: test bundled PostgreSQL metrics image +templates: + - charts/postgresql/templates/primary/statefulset.yaml +tests: + - it: should use the legacy exporter image when metrics are enabled + values: + - ../values.lint.yaml + set: + postgresql.metrics.enabled: true + asserts: + - equal: + path: spec.template.spec.containers[1].image + value: docker.io/bitnamilegacy/postgres-exporter:0.16.0-debian-12-r8 + template: charts/postgresql/templates/primary/statefulset.yaml diff --git a/charts/sourcebot/tests/redis_metrics_image_test.yaml b/charts/sourcebot/tests/redis_metrics_image_test.yaml new file mode 100644 index 0000000..e8d9fb3 --- /dev/null +++ b/charts/sourcebot/tests/redis_metrics_image_test.yaml @@ -0,0 +1,18 @@ +suite: test bundled Redis metrics image +templates: + - charts/redis/templates/configmap.yaml + - charts/redis/templates/health-configmap.yaml + - charts/redis/templates/scripts-configmap.yaml + - charts/redis/templates/secret.yaml + - charts/redis/templates/primary/application.yaml +tests: + - it: should use the legacy exporter image when metrics are enabled + values: + - ../values.lint.yaml + set: + redis.metrics.enabled: true + asserts: + - equal: + path: spec.template.spec.containers[1].image + value: docker.io/bitnamilegacy/redis-exporter:1.67.0-debian-12-r7 + template: charts/redis/templates/primary/application.yaml diff --git a/charts/sourcebot/values.yaml b/charts/sourcebot/values.yaml index feac102..50042e7 100644 --- a/charts/sourcebot/values.yaml +++ b/charts/sourcebot/values.yaml @@ -343,6 +343,11 @@ postgresql: # -- Overwrite default repository of helm chart to point to non-paid bitnami images repository: bitnamilegacy/postgresql + metrics: + image: + # -- Overwrite default metrics exporter repository to point to non-paid bitnami images + repository: bitnamilegacy/postgres-exporter + # Authentication configuration auth: # -- Username to connect to PostgreSQL @@ -391,6 +396,11 @@ redis: # -- Overwrite default repository of helm chart to point to non-paid bitnami images repository: bitnamilegacy/valkey + metrics: + image: + # -- Overwrite default metrics exporter repository to point to non-paid bitnami images + repository: bitnamilegacy/redis-exporter + # Authentication configuration auth: # -- Username for Redis connection From 796181cbb41483bc4461f9c71f442f827783f423 Mon Sep 17 00:00:00 2001 From: Brendan Kellam Date: Wed, 2 Sep 2026 18:14:28 -0700 Subject: [PATCH 2/2] chore: update changelog --- CHANGELOG.md | 3 +++ 1 file changed, 3 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 4c74940..5def086 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -7,6 +7,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 ## [Unreleased] +### Fixed +- PostgreSQL and Redis metrics exporters now use Bitnami Legacy images when enabled. [#136](https://github.com/sourcebot-dev/sourcebot-helm-chart/pull/136) + ## [0.1.104] - 2026-08-27 ### Changed