[MediaWiki-commits] [Gerrit] operations/puppet[production]: Limit Electron memory usage to 2G

2017-06-13 Thread Dzahn (Code Review)
Dzahn has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/35 )

Change subject: Limit Electron memory usage to 2G
..


Limit Electron memory usage to 2G

In normal operation, Electron uses significantly less memory, but there
have been instances where it started to use up to 15G. Since memory on
SCB nodes is currently very tight (ORES using most of it), this impacted
other services. It is better to avoid such impacts. Ideally, the service
would exit & restart.

Bug: T167834
Change-Id: If199a0271b7d36e26a3dd90e9c130fd71eb78945
---
M modules/pdfrender/templates/initscripts/pdfrender.systemd.erb
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  jenkins-bot: Verified
  Dzahn: Looks good to me, approved



diff --git a/modules/pdfrender/templates/initscripts/pdfrender.systemd.erb 
b/modules/pdfrender/templates/initscripts/pdfrender.systemd.erb
index 598fdb4..a2c3f22 100644
--- a/modules/pdfrender/templates/initscripts/pdfrender.systemd.erb
+++ b/modules/pdfrender/templates/initscripts/pdfrender.systemd.erb
@@ -7,6 +7,7 @@
 [Service]
 # up ulimit -n a bit
 LimitNOFILE=1
+MemoryLimit=2G
 User=pdfrender
 Group=pdfrender
 Environment="NODE_PATH=/srv/deployment/electron-render/deploy/node_modules"

-- 
To view, visit https://gerrit.wikimedia.org/r/35
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If199a0271b7d36e26a3dd90e9c130fd71eb78945
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: GWicke 
Gerrit-Reviewer: Dzahn 
Gerrit-Reviewer: Filippo Giunchedi 
Gerrit-Reviewer: Giuseppe Lavagetto 
Gerrit-Reviewer: Halfak 
Gerrit-Reviewer: Ladsgroup 
Gerrit-Reviewer: Mobrovac 
Gerrit-Reviewer: Ppchelko 
Gerrit-Reviewer: jenkins-bot <>

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits


[MediaWiki-commits] [Gerrit] operations/puppet[production]: Limit Electron memory usage to 2G

2017-06-13 Thread GWicke (Code Review)
GWicke has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/35 )

Change subject: Limit Electron memory usage to 2G
..

Limit Electron memory usage to 2G

In normal operation, Electron uses significantly less memory, but there
have been instances where it started to use up to 15G. Since memory on
SCB nodes is currently very tight (ORES using most of it), this impacted
other services. It is better to avoid such impacts. Ideally, the service
would exit & restart.

Bug: T167834
Change-Id: If199a0271b7d36e26a3dd90e9c130fd71eb78945
---
M modules/pdfrender/templates/initscripts/pdfrender.systemd.erb
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/88/35/1

diff --git a/modules/pdfrender/templates/initscripts/pdfrender.systemd.erb 
b/modules/pdfrender/templates/initscripts/pdfrender.systemd.erb
index 598fdb4..a2c3f22 100644
--- a/modules/pdfrender/templates/initscripts/pdfrender.systemd.erb
+++ b/modules/pdfrender/templates/initscripts/pdfrender.systemd.erb
@@ -7,6 +7,7 @@
 [Service]
 # up ulimit -n a bit
 LimitNOFILE=1
+MemoryLimit=2G
 User=pdfrender
 Group=pdfrender
 Environment="NODE_PATH=/srv/deployment/electron-render/deploy/node_modules"

-- 
To view, visit https://gerrit.wikimedia.org/r/35
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If199a0271b7d36e26a3dd90e9c130fd71eb78945
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: GWicke 

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits