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

Set default securityContext for nextcloud-metrics #558

Open Tuxprogrammer opened 2 months ago

Tuxprogrammer commented 2 months ago

Sets best defaults for securityContext on nextcloud-metrics pod and container.

Pull Request

Description of the change

Add defaults to the deployment of nextcloud-metrics with selections for securityContext options on both pod and container.

Benefits

Clusters with higher security configurations can run nextcloud with metrics enabled.

Possible drawbacks

Unknown, if any.

Applicable issues

Additional information

Checklist

Tuxprogrammer commented 2 months ago

related to #356

jessebot commented 2 months ago

These defaults should be set in the values.yaml and then templated into the metrics/deployment.yaml. Could you please do that so that users can customize what gets applied? Also, please bump the helm chart version.