feat(charts/paperless-ngx): use official container images for postgres and redis per default instead of bitnami images #61
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this PR does / why we need it
Use the official redis and postgres container image instead of deprecated bitnami container images.
Bitnami changed their image catalog and how they will publish their secure images. Duo to this, secure images are no longer available for free. Bitnami still publishes the latest version for development only, and old images will still be available under
docker.io/bitnamilegacy. Images likedocker.io/bitnami/redis:7.2.5-debian-12-r0which are referenced from the bitnami redis helm chart are no longer available. The old bitnami images are still available underdocker.io/bitnamilegacy/redis:7.2.5-debian-12-r0, but they will not publish new tags.Special notes for your reviewer
Postgres additionally needs an extra volume mount at
/var/run/postgresqlsince this is where postgres puts it pid/socket in order to keep a read only root filesystem. As an alternative the read only root filesystem could be set to false withI tested the chart with updated images, which worked well, but eventually it is also necessary to set
to suppress the warning introduced with bitnami/charts#30850, which I could not reproduce.
Additionally you could take a look at this commit of the authentik helm chart, which alos replaced the redis and postgres bitnami container images with official images.