Make nginx timeout configurable in helm chart

Signed-off-by: Dmitriy Barbul <dimabarbul@gmail.com>
eclipse-ditto · May 14, 2024 · f02d70d · f02d70d
1 parent d98cfa6
commit f02d70d
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 6 deletions.
diff --git a/deployment/helm/ditto/Chart.yaml b/deployment/helm/ditto/Chart.yaml
@@ -16,7 +16,7 @@ description: |
   A digital twin is a virtual, cloud based, representation of his real world counterpart
   (real world “Things”, e.g. devices like sensors, smart heating, connected cars, smart grids, EV charging stations etc).
 type: application
-version: 3.5.4  # chart version is effectively set by release-job
+version: 3.5.5-0  # chart version is effectively set by release-job
 appVersion: 3.5.4
 keywords:
   - iot-chart

diff --git a/deployment/helm/ditto/templates/nginx-configmap.yaml b/deployment/helm/ditto/templates/nginx-configmap.yaml
@@ -30,10 +30,9 @@ data:
       default_type application/json;
       include mime.types;
     
-      # timeouts are configured slightly higher than ditto-eclipse-ditto-gateway read-timeout of 60 seconds
-      proxy_connect_timeout 70; # seconds, default: 60
-      proxy_send_timeout 70; # seconds, default: 60
-      proxy_read_timeout 70; # seconds, default: 60
+      proxy_connect_timeout {{ .Values.nginx.config.timeout }}; # seconds, default: 60
+      proxy_send_timeout {{ .Values.nginx.config.timeout }}; # seconds, default: 60
+      proxy_read_timeout {{ .Values.nginx.config.timeout }}; # seconds, default: 60
       # will try another upstream if an error or timeout occurred during the connection
       # or if the upstream returns 502 response
       proxy_next_upstream error timeout http_502;
@@ -42,7 +41,7 @@ data:
       # will try for max. 20s to find another upstream to connect to
       proxy_next_upstream_timeout 20;
 
-      send_timeout 70; # seconds, default: 60
+      send_timeout {{ .Values.nginx.config.timeout }}; # seconds, default: 60
     
       client_header_buffer_size 8k; # allow longer URIs + headers (default: 1k)
       large_client_header_buffers 4 16k;

diff --git a/deployment/helm/ditto/values.yaml b/deployment/helm/ditto/values.yaml
@@ -1679,6 +1679,9 @@ nginx:
     workerProcesses: 4
     # workerProcesses the 'events' 'worker_connections' option for nginx to use
     workerConnections: 1024
+    # the 'proxy_connect_timeout', 'proxy_send_timeout', 'proxy_read_timeout' and 'send_timeout' options for nginx
+    # timeouts are configured slightly higher than ditto-eclipse-ditto-gateway read-timeout
+    timeout: 70
 
 ## ----------------------------------------------------------------------------
 ## Ditto UI configuration