Skip to content

Commit

Permalink
chore(mpp): adjust the templates to current limits
Browse files Browse the repository at this point in the history
During the redeployment (after #558), after fixing the hard-coded
project value, it has been discovered that we are hitting limits
of the resources we're using on MP+, so there were some ad-hoc
adjustments done on the fly.

Sync those adjustments to reflect the currently deployed
requests/limits.

Signed-off-by: Matej Focko <[email protected]>
  • Loading branch information
mfocko committed Apr 18, 2024
1 parent bedef20 commit 9e98437
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion openshift/packit-service.yml.j2
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ spec:
# If you see '/usr/bin/run_httpd.sh: line 16: Killed alembic upgrade head'
# you have to temporarily increase (in webUI/console) the limit
# and once the alembic upgrade passes, revert.
memory: "{{ '4Gi' if project == 'packit--prod' else '512Mi' }}"
memory: "{{ '1Gi' if project == 'packit--prod' else '512Mi' }}"
cpu: "200m"
# In TLS world, hostname needs to match whatever is set in the cert
# in our cause, k8s is doing here something like curl https://172.15.2.4:8443/api/healthz/
Expand Down
4 changes: 2 additions & 2 deletions openshift/postgres.yml.j2
Original file line number Diff line number Diff line change
Expand Up @@ -58,12 +58,12 @@ spec:
# - /usr/libexec/check-container
resources:
requests:
memory: "{{ '2Gi' if project == 'packit--prod' else '256Mi' }}"
memory: "{{ '1Gi' if project == 'packit--prod' else '256Mi' }}"
cpu: "30m"
limits:
# The shared_buffers & effective_cache_size values are set by the container
# based on this limit, so keep the limit reasonably high.
memory: "{{ '4Gi' if project == 'packit--prod' else '512Mi' }}"
memory: "{{ '1536Mi' if project == 'packit--prod' else '512Mi' }}"
# Not utilized most of the time, but useful during migrations and for some queries
cpu: "1"
volumeMounts:
Expand Down

0 comments on commit 9e98437

Please sign in to comment.