Skip to content

[TRNT-4376] Move PostgreSQL config to global#192

Merged
antgamdia merged 7 commits into
mainfrom
TRNT-4376-update-postgres
May 13, 2026
Merged

[TRNT-4376] Move PostgreSQL config to global#192
antgamdia merged 7 commits into
mainfrom
TRNT-4376-update-postgres

Conversation

@antgamdia

@antgamdia antgamdia commented May 12, 2026

Copy link
Copy Markdown
Contributor

Description

After we discovered some misalignment in the versions used for development (see trento-project/wanda#712 and trento-project/web#4290), we should consistently use a version for our database dependencies.

This PR chooses 17.5 as it is the default version being shipped in the OS version we base our images on. As of today, for 15 SP7 images, the PostgreSQL version is 17.5 (source). This PR, therefore, sets this version globally.

Edit: after some discussions, this PR simply prepares the chart for a future update; not does not perform the actual update.

To make it happen, we moved the image config to the globals section and, just for backwards compatibility, we set all the subcharts' values to this new version to avoid further inconsistencies.

Related # TRNT-4376

How was this tested?

CI

antgamdia added 5 commits May 12, 2026 17:50
Signed-off-by: Antonio Gamez Diaz <antonio.gamez@suse.com>
Signed-off-by: Antonio Gamez Diaz <antonio.gamez@suse.com>
Signed-off-by: Antonio Gamez Diaz <antonio.gamez@suse.com>
Signed-off-by: Antonio Gamez Diaz <antonio.gamez@suse.com>
Signed-off-by: Antonio Gamez Diaz <antonio.gamez@suse.com>
@antgamdia antgamdia changed the title [TRNT-4376] Update PostgreSQL version to 15.5 [TRNT-4376] Update PostgreSQL version to 17.5 May 12, 2026
@antgamdia antgamdia requested review from arbulu89 and balanza May 12, 2026 16:19

@arbulu89 arbulu89 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!
Important to test these changes really carefully as we are really close to release. Specially, a migration for an already deployed k3s with an older postgresql version. We need to be sure that the data that existed before is properly set in the new pod.
@antgamdia please, run those tests.

@antgamdia

Copy link
Copy Markdown
Contributor Author

Important to test these changes really carefully as we are really close to release.

Yes, that's very true... maybe we can even postpone this change.

Specially, a migration for an already deployed k3s with an older postgresql version. We need to be sure that the data that existed before is properly set in the new pod.

Good idea. Ideally, we should have automated upgrade tests in our helm chart CI, though. But, yep, let me double-check manually. Thanks for the heads-up!

antgamdia added 2 commits May 13, 2026 13:41
Signed-off-by: Antonio Gamez Diaz <antonio.gamez@suse.com>
Signed-off-by: Antonio Gamez Diaz <antonio.gamez@suse.com>
@antgamdia antgamdia changed the title [TRNT-4376] Update PostgreSQL version to 17.5 [TRNT-4376] Move PostgreSQL config to global May 13, 2026
@antgamdia

Copy link
Copy Markdown
Contributor Author

@arbulu89 , I've tracked the pg update separately; however, some of the changes herein performed can still be useful.
I've moved the PG image to global but I've kept the version set to 14, so that there are no functional changes.

> helm template .\charts\trento-server\  | grep "image:" | awk '{print $2}' | tr -d '"' | sort -u

busybox
docker.io/alpine:3.19
docker.io/rabbitmq:3.12.6-management-alpine
ghcr.io/trento-project/checks:1.2.0
ghcr.io/trento-project/trento-wanda:2.0.0
ghcr.io/trento-project/trento-web:3.0.0
k8s.gcr.io/kube-state-metrics/kube-state-metrics:v2.3.0
prom/pushgateway:v1.4.2
quay.io/prometheus/node-exporter:v1.3.0
quay.io/prometheus/prometheus:v2.53.1
registry.k8s.io/kubectl:v1.33.3
registry.suse.com/suse/nginx:1.27
registry.suse.com/suse/postgres:14

@arbulu89 arbulu89 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!
Green light from my side.
Now, updating to a new version will be trivial!

@antgamdia antgamdia merged commit b880e61 into main May 13, 2026
5 checks passed
@antgamdia antgamdia deleted the TRNT-4376-update-postgres branch May 13, 2026 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants