Add `metrics.securityContext` and `metrics.podSecurityContext` - Githubissues

nextcloud / helm

A community maintained helm chart for deploying Nextcloud on Kubernetes.

GNU Affero General Public License v3.0

334 stars 269 forks source link

Add `metrics.securityContext` and `metrics.podSecurityContext` #609

Closed jessebot closed 3 months ago

jessebot commented 3 months ago

Description of the change

Allow users to set the securityContext on both the metrics container and the pod.

Benefits

More security is good :)

Possible drawbacks

can't think of any, but happy to discuss

Applicable issues

closes #558 as it's virtually the same PR, but with the changes requested in the review as the developer is too busy right now
partially addresses #356 as it also covers some of this, but the developer is too busy right now

Additional information

n/a

Checklist

[x] I have read the CONTRIBUTING.md doc.
[x] DCO has been signed off on the commit.
[x] Chart version bumped in Chart.yaml according to semver.
[x] (optional) Parameters are documented in the README.md