Skip to content

Commit

Permalink
docs: update security context in docs (#1773)
Browse files Browse the repository at this point in the history
Rewrites the examples in accordance to #1768. Security context overrides are not
needed in a working k8s setup and will only confuse if left in examples uncommented
  • Loading branch information
theSuess authored Nov 25, 2024
1 parent c71f027 commit ae940eb
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 11 deletions.
14 changes: 7 additions & 7 deletions examples/grafana_deployment/resources.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,6 @@ metadata:
dashboards: "grafana"
spec:
config:
log:
mode: "console"
auth:
disable_login_form: "false"
security:
admin_user: root
admin_password: secret
Expand All @@ -20,9 +16,13 @@ spec:
spec:
containers:
- name: grafana
image: grafana/grafana:9.4.3
securityContext:
allowPrivilegeEscalation: true
readOnlyRootFilesystem: false
# Customize this in case your volume provider needs specific UIDs/GIDs
runAsUser: 1001
runAsGroup: 1001
runAsNonRoot: true
allowPrivilegeEscalation: false
capabilities:
drop: ["ALL"]
readinessProbe:
failureThreshold: 3
4 changes: 0 additions & 4 deletions examples/persistent_volume/resources.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -28,10 +28,6 @@ spec:
spec:
containers:
- name: grafana
image: grafana/grafana:9.4.3
securityContext:
allowPrivilegeEscalation: true
readOnlyRootFilesystem: false
readinessProbe:
failureThreshold: 3
volumes:
Expand Down

0 comments on commit ae940eb

Please sign in to comment.