Skip to content

Commit

Permalink
feat: securityContext more flexible set, fix higress-group#289
Browse files Browse the repository at this point in the history
  • Loading branch information
OnlyPiglet committed Feb 25, 2024
1 parent e5aad32 commit 539ed9f
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 9 deletions.
5 changes: 1 addition & 4 deletions helm/templates/prometheus.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -134,11 +134,8 @@ spec:
memory: {{ $config.resources.limits.memory }}
restartPolicy: Always
serviceAccountName: {{ $appName }}
{{- if .Values.o11y.prometheus.securityContext.enabled }}
securityContext:
fsGroup: {{ .Values.o11y.prometheus.securityContext.fsGroup }}
runAsUser: {{ .Values.o11y.prometheus.securityContext.runAsUser }}
{{- end }}
{{- toYaml .Values.o11y.prometheus.securityContext | nindent 8 }}
volumes:
- name: data
persistentVolumeClaim:
Expand Down
7 changes: 2 additions & 5 deletions helm/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -80,9 +80,6 @@ o11y:
limits:
cpu: 500m
memory: 2Gi
securityContext:
enabled: true
runAsUser: 0
fsGroup: 0
securityContext: {}
pvc:
rwxSupported: true
rwxSupported: true

0 comments on commit 539ed9f

Please sign in to comment.