allow zotero container nodejs server to define the amount of heap used instead of the fixed limit of 1.7Gi
Open, NormalPublic

Description

we had a briefly outage of one minute when zotero was unresponsive, our current believe is that nodejs heap size was filled up and hence the server wasn't responding

https://grafana.wikimedia.org/d/000000620/xxxx-zotero-debugging-kubernetes?orgId=1&from=1547060027804&to=1547062295521

In order to change that we need to change the docker image to allow an adjustable heap size and modify also the helm chart to allow to define the max heap size which should be Pod Mem Limit size - X, where X is the amount of RAM the nodejs app is using besides heap.

fselles created this task.Thu, Jan 10, 12:26 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptThu, Jan 10, 12:26 PM
jijiki added a subscriber: jijiki.

Change 483395 had a related patch set uploaded (by Fsero; owner: Fsero):
[mediawiki/services/zotero@master] Added NODE_V8_HEAP_SIZE_IN_MB env variable and made node honor it.

https://gerrit.wikimedia.org/r/483395

Change 483398 had a related patch set uploaded (by Fsero; owner: Fsero):
[operations/deployment-charts@master] Added defaults of node heap size to match the new one introduced.

https://gerrit.wikimedia.org/r/483398

Change 483395 merged by jenkins-bot:
[mediawiki/services/zotero@master] Added NODE_V8_HEAP_SIZE_IN_MB env variable and made node honor it.

https://gerrit.wikimedia.org/r/483395

fselles moved this task from Backlog to Radar on the serviceops board.Mon, Jan 14, 10:08 AM
fselles moved this task from Radar to Backlog on the serviceops board.
fselles moved this task from Backlog to Doing on the serviceops board.
akosiaris triaged this task as Normal priority.Mon, Jan 14, 3:06 PM
akosiaris added a subscriber: akosiaris.

An image that allows overriding the node heap has now been deployed.