Skip to content

Commit c84d91e

Browse files
Merge pull request #633 from snyk/fix/nodejs-memory-enhancements
fix: memory enhancements for Node.js app
2 parents 3aee6bb + cdb1261 commit c84d91e

File tree

2 files changed

+12
-0
lines changed

2 files changed

+12
-0
lines changed

snyk-monitor/templates/deployment.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,9 @@ spec:
7575
value: {{ .Values.no_proxy }}
7676
- name: LOG_LEVEL
7777
value: {{ .Values.log_level }}
78+
{{- with .Values.envs }}
79+
{{- toYaml . | trim | nindent 10 -}}
80+
{{ end }}
7881
resources:
7982
requests:
8083
cpu: {{ .Values.requests.cpu }}

snyk-monitor/values.yaml

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,15 @@ pvc:
4141
enabled: false
4242
name: 'snyk-monitor-pvc'
4343

44+
# Node.js in-container process memory enhancements
45+
envs:
46+
- name: V8_MAX_OLD_SPACE_SIZE
47+
value: '2048'
48+
- name: UV_THREADPOOL_SIZE
49+
value: '24'
50+
- name: NODE_OPTIONS
51+
value: --max_old_space_size=2048
52+
4453
# CPU/Mem requests and limits for snyk-monitor
4554
requests:
4655
cpu: '250m'

0 commit comments

Comments
 (0)