Skip to content

Commit 810ecb4

Browse files
Merge pull request #8 from sourcebot-dev/bkellam/non-root
chore: set podSecurityContext to non-root by default
2 parents 689f58c + 3a44bbf commit 810ecb4

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

charts/sourcebot/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ Sourcebot is a self-hosted tool that helps you understand your codebase.
9090
| sourcebot.podDisruptionBudget.enabled | bool | `true` | Enable Pod Disruption Budget |
9191
| sourcebot.podDisruptionBudget.maxUnavailable | int | `1` | Maximum number of pods that can be unavailable |
9292
| sourcebot.podDisruptionBudget.minAvailable | int | `1` | Minimum number of pods that must be available |
93-
| sourcebot.podSecurityContext | object | `{}` | Set the pod-level security context |
93+
| sourcebot.podSecurityContext | object | `{"fsGroup":1500,"runAsGroup":1500,"runAsNonRoot":true,"runAsUser":1500}` | Set the pod-level security context |
9494
| sourcebot.priorityClassName | string | `""` | Set the priority class name for pods See: https://kubernetes.io/docs/concepts/scheduling-eviction/pod-priority-preemption/ |
9595
| sourcebot.readinessProbe.failureThreshold | int | `5` | Number of consecutive failures before marking the container as not ready |
9696
| sourcebot.readinessProbe.httpGet | object | `{"path":"/api/health","port":"http"}` | Http GET request to check if the container is ready |

charts/sourcebot/values.yaml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -93,11 +93,11 @@ sourcebot:
9393
automount: false
9494

9595
# -- Set the pod-level security context
96-
podSecurityContext: {}
97-
# runAsUser: 1000
98-
# runAsGroup: 1000
99-
# runAsNonRoot: true
100-
# fsGroup: 1000
96+
podSecurityContext:
97+
runAsUser: 1500
98+
runAsGroup: 1500
99+
fsGroup: 1500
100+
runAsNonRoot: true
101101

102102
# -- Set the container-level security context
103103
containerSecurityContext: {}

0 commit comments

Comments
 (0)