[MediaWiki-commits] [Gerrit] operations/puppet[production]: PDF Render: Lower the concurrency to 4

2017-08-16 Thread Alexandros Kosiaris (Code Review) 
Alexandros Kosiaris has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/372156 )

Change subject: PDF Render: Lower the concurrency to 4
..


PDF Render: Lower the concurrency to 4

We have been having problems with the service's restart procedure where
it would block during the window-creation process. However, that usually
happens after at least 4 windows have been created. Since our current
rate is 2 reqs/sec and the service is deployed on all SCB hosts, it
should be safe to lower the concurrency to 4 to minimise the restart
hangs.

Bug: T159922
Change-Id: I19d2b00ac01b64d1e89e3b885c9ea6ff42f33976
---
M modules/role/manifests/pdfrender.pp
1 file changed, 1 insertion(+), 1 deletion(-)

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



diff --git a/modules/role/manifests/pdfrender.pp 
b/modules/role/manifests/pdfrender.pp
index 21cf388..657def0c 100644
--- a/modules/role/manifests/pdfrender.pp
+++ b/modules/role/manifests/pdfrender.pp
@@ -9,7 +9,7 @@
 
 class { '::pdfrender':
 port=> $port,
-no_browsers => 8,
+no_browsers => 4,
 running => $is_active,
 }
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I19d2b00ac01b64d1e89e3b885c9ea6ff42f33976
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Mobrovac 
Gerrit-Reviewer: Alexandros Kosiaris 
Gerrit-Reviewer: Filippo Giunchedi 
Gerrit-Reviewer: GWicke 
Gerrit-Reviewer: Giuseppe Lavagetto 
Gerrit-Reviewer: Muehlenhoff 
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]: PDF Render: Lower the concurrency to 4

2017-08-16 Thread Mobrovac (Code Review) 
Mobrovac has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/372156 )

Change subject: PDF Render: Lower the concurrency to 4
..

PDF Render: Lower the concurrency to 4

We have been having problems with the service's restart procedure where
it would block during the window-creation process. However, that usually
happens after at least 4 windows have been created. Since our current
rate is 2 reqs/sec and the service is deployed on all SCB hosts, it
should be safe to lower the concurrency to 4 to minimise the restart
hangs.

Bug: T159922
Change-Id: I19d2b00ac01b64d1e89e3b885c9ea6ff42f33976
---
M modules/role/manifests/pdfrender.pp
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/56/372156/1

diff --git a/modules/role/manifests/pdfrender.pp 
b/modules/role/manifests/pdfrender.pp
index 21cf388..657def0c 100644
--- a/modules/role/manifests/pdfrender.pp
+++ b/modules/role/manifests/pdfrender.pp
@@ -9,7 +9,7 @@
 
 class { '::pdfrender':
 port=> $port,
-no_browsers => 8,
+no_browsers => 4,
 running => $is_active,
 }
 

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

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

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