Merge pull request #326 from opencost/claude/nginx-proxy-timeouts-helm-YErRj

Add configurable nginx proxy timeouts to helm chart

Author: ameijer

charts/opencost/Chart.yaml

@@ -9,7 +9,7 @@ keywords:
 - finops
 - monitoring
 - opencost
-version: 2.5.2
+version: 2.5.3
 maintainers:
   - name: jessegoodier
   - name: toscott

charts/opencost/templates/configmap-frontend.yaml

@@ -80,9 +80,9 @@ data:
             return 200 'OK';
         }
         location {{ .Values.opencost.ui.uiPath | trimSuffix "/" }}/model/ {
-            proxy_connect_timeout       180;
-            proxy_send_timeout          180;
-            proxy_read_timeout          180;
+            proxy_connect_timeout       {{ .Values.opencost.ui.nginx.proxyConnectTimeout }};
+            proxy_send_timeout          {{ .Values.opencost.ui.nginx.proxySendTimeout }};
+            proxy_read_timeout          {{ .Values.opencost.ui.nginx.proxyReadTimeout }};
             proxy_pass http://model/;
             proxy_redirect off;
             proxy_http_version 1.1;

charts/opencost/values.yaml

@@ -538,6 +538,15 @@ opencost:
     # set to true to set upstream to use <service>.<namespace>.svc.cluster.local instead of just <service>.<namespace>
     useDefaultFqdn: false
 
+    # -- Nginx proxy timeout settings (in seconds)
+    nginx:
+      # -- Timeout for establishing a connection with the proxied server
+      proxyConnectTimeout: 180
+      # -- Timeout for transmitting a request to the proxied server
+      proxySendTimeout: 180
+      # -- Timeout for reading a response from the proxied server
+      proxyReadTimeout: 180
+
     # Set the model fqdn to use for the upstream
     # modelFqdn: opencost.opencost.svc.cluster.local:9003