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

`nextcloud.existingSecret` doesn't cause `NEXTCLOUD_AUTH_TOKEN` to be set #508

Open felixscheinost opened 5 months ago

felixscheinost commented 5 months ago

Describe your Issue

There is a or .Values.nextcloud.existingSecret.enabled missing here: https://github.com/nextcloud/helm/blob/3b7b2f24ca5c8da56b4ad29b80e66ded4a6da69a/charts/nextcloud/templates/metrics-deployment.yaml#L41

jessebot commented 1 month ago

That makes sense! You can submit a PR for this, just make sure the bump the version in Chart.yaml and this seems like an easy approval and merge :) If you, or others, submit a PR for this, please link back to this issue by including closes #508 in the PR description. Further details about contributing can be found here.