[arvados] updated: 2.6.0-194-g61a43214b

git repository hosting git at public.arvados.org
Thu May 25 18:59:21 UTC 2023


Summary of changes:
 doc/admin/metrics.html.textile.liquid                                | 2 +-
 .../config_examples/multi_host/aws/pillars/nginx_passenger.sls       | 5 +++--
 2 files changed, 4 insertions(+), 3 deletions(-)

       via  61a43214b114c4dddaca37f20cd14f8c5544558e (commit)
       via  a85f4a2b994abe84d1244065c7c8c0dfa362eef9 (commit)
      from  e8a6dc77fdc28cb4df6a5b164b14f0f6549f2112 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 61a43214b114c4dddaca37f20cd14f8c5544558e
Author: Lucas Di Pentima <lucas.dipentima at curii.com>
Date:   Thu May 25 15:58:53 2023 -0300

    20474: Updates documentation stating that RailsAPI supports /metrics endpoint.
    
    Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas.dipentima at curii.com>

diff --git a/doc/admin/metrics.html.textile.liquid b/doc/admin/metrics.html.textile.liquid
index b140bcc1b..79d895ae4 100644
--- a/doc/admin/metrics.html.textile.liquid
+++ b/doc/admin/metrics.html.textile.liquid
@@ -31,7 +31,7 @@ When configuring Prometheus, use a @bearer_token@ or @bearer_token_file@ option
 
 table(table table-bordered table-condensed table-hover).
 |_. Component|_. Metrics endpoint|
-|arvados-api-server||
+|arvados-api-server|✓|
 |arvados-controller|✓|
 |arvados-dispatch-cloud|✓|
 |arvados-dispatch-lsf|✓|

commit a85f4a2b994abe84d1244065c7c8c0dfa362eef9
Author: Lucas Di Pentima <lucas.dipentima at curii.com>
Date:   Thu May 25 15:52:32 2023 -0300

    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 at 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 bd1aacffa..7bbf9ae61 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

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list