nextcloud / helm

A community maintained helm chart for deploying Nextcloud on Kubernetes.
GNU Affero General Public License v3.0
295 stars 258 forks source link

Feature: Pin a specific `nginx` tag #509

Open felixscheinost opened 5 months ago

felixscheinost commented 5 months ago

Description of the change

Currently nginx:alpine is used which is bad practice, as the image will never be updated after the initial pull. I suggest that rather a specific nginx version should be used, which would be updated regularly.

Benefits

Possible drawbacks

provokateurin commented 5 months ago

We could also change the nginx.image.pullPolicy to Always. Either as a temporary workaround or a permanent. I agree though that we should point to a specific version, so let's do it the proper way.

jessebot commented 2 months ago

I second a specific version that gets updated somehow via dependabot or renovatebot. Marked as good first issue for anyone who knows helm well enough to get this done.