sonatype / nxrm3-helm-repository

Helm3 charts for Nexus Repository Manager
Other
33 stars 37 forks source link

INT-7432 security context for openshift #34

Closed jflinchbaugh closed 1 year ago

jflinchbaugh commented 1 year ago

OpenShift wants more securityContext settings. These settings are also fine in minikube.

Reproduction:

JIRA: https://issues.sonatype.org/browse/INT-7432 Build: https://jenkins.ci.sonatype.dev/job/nxrm/job/Nexus%20Repository%20Manager%203%20Helm3%20Charts/job/Feature%20Snapshot%20Builds/job/INT-7432-securityContext-for-openshift/

jflinchbaugh commented 1 year ago

Its hard to know from the docs about - RuntimeDefault

The warning message offers "RuntimeDefault" or "Localhost" as acceptable values. "RuntimeDefault" implies the cluster admin can set it for themselves, so more flexible when we don't know how people may have things configured. (and it continued to work without warning.)

jflinchbaugh commented 1 year ago

Feel free to merge this when you like it, or let me know when I can merge it.

bobotimi commented 1 year ago

Feel free to merge this when you like it, or let me know when I can merge it.

+1 Yep, go for it