Skip to content

Commit

Permalink
Set Set cpu/memory requests/limits for tekton-results-api init contai…
Browse files Browse the repository at this point in the history
…ners

Init containers are harder to figure out the required resources as they
do not run long enough to have metrics collected. Both container worked
with 5m/32Mi in a dev environment. Put the limits higher to make sure
they works in production environment even though the nature of the work
done by both should not differ from dev to production environment.

PLNSRVCE-1476

Signed-off-by: Hugo Arès <[email protected]>
  • Loading branch information
hugares authored and Roming22 committed Oct 20, 2023
1 parent f74b260 commit 47c14a4
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,13 @@ spec:
subPath: s3-cert.crt
- name: tmp-mc-volume
mountPath: /tmp
resources:
requests:
cpu: 5m
memory: 32Mi
limits:
cpu: 100m
memory: 128Mi
securityContext:
readOnlyRootFilesystem: true
runAsNonRoot: true
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -34,5 +34,12 @@ spec:
secretKeyRef:
name: tekton-results-database
key: db.name
resources:
requests:
cpu: 5m
memory: 32Mi
limits:
cpu: 100m
memory: 128Mi
securityContext:
readOnlyRootFilesystem: true

0 comments on commit 47c14a4

Please sign in to comment.