Skip to content

Commit e2cc809

Browse files
Include default resource requests for garbage collector (#160)
### Changelog <!-- Write a one-sentence summary of the user-impacting change (API, UI/UX, performance, etc) that could appear in a changelog. Write "None" if there is no user-facing change --> - primary-site: include default resource requests for garbage collector ### Docs <!-- Link to a Docs PR, tracking ticket in Linear, OR write "None" if no documentation changes are needed. --> None ### Description Some clouds don't give pods default resources, so make sure they're there just in case. Fixes FIRE-240.
1 parent 6bba181 commit e2cc809

File tree

2 files changed

+14
-0
lines changed

2 files changed

+14
-0
lines changed

charts/primary-site/templates/cronjobs/garbage-collector.yaml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,13 @@ spec:
3333
containers:
3434
- name: garbage-collector
3535
image: {{ .Values.garbageCollector.deployment.image }}:{{ .Chart.AppVersion }}
36+
resources:
37+
requests:
38+
cpu: {{ .Values.garbageCollector.deployment.resources.requests.cpu }}
39+
memory: {{ .Values.garbageCollector.deployment.resources.requests.memory }}
40+
limits:
41+
cpu: {{ .Values.garbageCollector.deployment.resources.limits.cpu }}
42+
memory: {{ .Values.garbageCollector.deployment.resources.limits.memory }}
3643
securityContext:
3744
readOnlyRootFilesystem: true
3845
capabilities:

charts/primary-site/values.yaml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -243,6 +243,13 @@ garbageCollector:
243243
extraVolumeMounts: []
244244
env: []
245245
nodeSelectors: {}
246+
resources:
247+
requests:
248+
cpu: 250m
249+
memory: 250Mi
250+
limits:
251+
cpu: 250m
252+
memory: 250Mi
246253
serviceAccount:
247254
enabled: false
248255
annotations: {}

0 commit comments

Comments
 (0)