From 3b1a39c2d0bee062df51a0a67d6277687728c365 Mon Sep 17 00:00:00 2001
From: shlomiko <shlomiko@rafael.co.il>
Date: Thu, 10 Sep 2026 15:20:57 +0300
Subject: [PATCH] fix(helm): configurable imagePullPolicy and task polling
 defaults

- add a top-level imagePullPolicy value (default IfNotPresent) and use it
  in the deployment instead of the hardcoded Always
- default TASK.pollingTime and heartbeat.intervalMs to 3000, matching the
  raster-export and raster-ingestion umbrella overrides
- default mclabels.prometheus.port to 9500, the port the service exposes
  metrics on

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
---
 helm/templates/deployment.yaml | 2 +-
 helm/values.yaml               | 7 ++++---
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/helm/templates/deployment.yaml b/helm/templates/deployment.yaml
index 54555aa..9d28346 100644
--- a/helm/templates/deployment.yaml
+++ b/helm/templates/deployment.yaml
@@ -128,7 +128,7 @@ spec:
           envFrom:
             - configMapRef:
                 name: {{ $configmapName }}
-          imagePullPolicy: "Always"
+          imagePullPolicy: {{ .Values.imagePullPolicy }}
           ports:
             - name: http
               containerPort: 8080
diff --git a/helm/values.yaml b/helm/values.yaml
index a1f3806..028b772 100644
--- a/helm/values.yaml
+++ b/helm/values.yaml
@@ -41,7 +41,7 @@ mclabels:
   gisDomain: raster
   prometheus:
     enabled: true
-    port: 8080
+    port: 9500
 
 enabled: true
 environment: development
@@ -50,6 +50,7 @@ initialDelaySeconds: 60
 nodePort: 30011
 resetOnConfigChange: true
 isExporter: false
+imagePullPolicy: IfNotPresent
 
 cloudProvider:
   dockerRegistryUrl: 'my-registry'
@@ -108,10 +109,10 @@ env:
         taskType: 'tilesMerging'
       - jobType: 'Ingestion_New'
         taskType: 'tilesMerging'
-    pollingTime: 1000
+    pollingTime: 3000
     maxAttempts: 3
   heartbeat:
-    intervalMs: 300
+    intervalMs: 3000
   HTTP:
     retries: 2
   logging: