Skip to content

Commit

Permalink
Merge pull request #105 from m00nyONE/patch-1
Browse files Browse the repository at this point in the history
fix: add securityContext to allow running as non root user
  • Loading branch information
axel7083 authored Dec 22, 2023
2 parents 7453030 + d402eed commit 430b07f
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions charts/cluster-secret/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,8 @@ spec:
app: clustersecret
{{- include "cluster-secret.selectorLabels" . | nindent 8 }}
spec:
securityContext:
runAsUser: 100 # 100 is set by the container and can NOT be changed here - this would result in a getpwuid() error
containers:
- env:
- name: KUBERNETES_CLUSTER_DOMAIN
Expand All @@ -34,6 +36,8 @@ spec:
image: {{ .Values.clustersecret.clustersecret.image.repository }}:{{ .Values.clustersecret.clustersecret.image.tag
| default .Chart.AppVersion }}
name: clustersecret
securityContext:
runAsUser: 100 # 100 is set by the container and can NOT be changed here - this would result in a getpwuid() error
livenessProbe:
httpGet:
path: /healthz
Expand Down

0 comments on commit 430b07f

Please sign in to comment.