Dzahn has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/372426 )

Change subject: Gerrit: Also set minimum heap size
......................................................................


Gerrit: Also set minimum heap size

Ideally we could set this in gerrit.config alongside container.heapLimit
but this will totally do for now

Bug: T148478
Change-Id: I647e073dedf8700acc13e5d4cb3756b32ed35edd
---
M modules/gerrit/manifests/jetty.pp
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Paladox: Looks good to me, but someone else must approve
  jenkins-bot: Verified
  Gehel: Looks good to me, but someone else must approve
  Dzahn: Looks good to me, approved



diff --git a/modules/gerrit/manifests/jetty.pp 
b/modules/gerrit/manifests/jetty.pp
index 294457c..81fc29a 100644
--- a/modules/gerrit/manifests/jetty.pp
+++ b/modules/gerrit/manifests/jetty.pp
@@ -54,6 +54,7 @@
         '-XX:+UseGCLogFileRotation',
         '-XX:NumberOfGCLogFiles=10',
         '-XX:GCLogFileSize=2M',
+        "-Xms ${heap_limit}",
         # 
'-Dlog4j.configuration=file:///var/lib/gerrit2/review_site/etc/log4j.properties',
     ]
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I647e073dedf8700acc13e5d4cb3756b32ed35edd
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Chad <[email protected]>
Gerrit-Reviewer: Dzahn <[email protected]>
Gerrit-Reviewer: Gehel <[email protected]>
Gerrit-Reviewer: Paladox <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to