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

Allow to set environment variables to Nginx container #566

Closed fmenabe closed 4 weeks ago

fmenabe commented 1 month ago

Pull Request

Description of the change

Add the nginx.extraEnv parameter allowing to pass environment variables to the Nginx container.

Benefits

This is for example useful to set the timezone of the Nginx container through TZ environment variable (cf #494 and https://github.com/nginxinc/docker-nginx/issues/149).

Possible drawbacks

None, the new parameter is optional.

Applicable issues

fixes #494

Additional information

Checklist

[X] DCO has been signed off on the commit.
[X] Chart version bumped in Chart.yaml according to semver.
[X] (optional) Variables are documented in the README.md

jessebot commented 4 weeks ago

approved, but waiting for tests to finish before merging