From: Lucas Di Pentima <lucas.dipentima@curii.com>
Date: Thu, 25 May 2023 18:52:32 +0000 (-0300)
Subject: 20474: Changes RailsAPI queue size to be 10% more than controller's.
X-Git-Tag: 2.7.0~101^2~1
X-Git-Url: https://git.arvados.org/arvados.git/commitdiff_plain/a85f4a2b994abe84d1244065c7c8c0dfa362eef9

20474: Changes RailsAPI queue size to be 10% more than controller's.

Also, adds comment on config file clarifying why we're setting up this
increased value.

Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas.dipentima@curii.com>
---

diff --git a/tools/salt-install/config_examples/multi_host/aws/pillars/nginx_passenger.sls b/tools/salt-install/config_examples/multi_host/aws/pillars/nginx_passenger.sls
index bd1aacffac..7bbf9ae618 100644
--- a/tools/salt-install/config_examples/multi_host/aws/pillars/nginx_passenger.sls
+++ b/tools/salt-install/config_examples/multi_host/aws/pillars/nginx_passenger.sls
@@ -24,8 +24,9 @@ nginx:
     passenger_max_pool_size: {{ "__CONTROLLER_NGINX_WORKERS__" or grains['num_cpus'] }}
     {%- set max_reqs = "__CONTROLLER_MAX_CONCURRENT_REQUESTS__" %}
     {%- if max_reqs != "" and max_reqs is number %}
-    # Default is 100
-    passenger_max_request_queue_size: {{ max_reqs + 5 }}
+    # Default is 100 -- Configuring this a bit higher than API.MaxConcurrentRequests
+    # to be able to handle /metrics requests even on heavy load situations.
+    passenger_max_request_queue_size: {{ (max_reqs * 1.1)|round|int }}
     {%- endif %}
 
   ### SERVER