refactor: move image configuration to global values for consistent container image management

Author: JannikZed

charts/saleor/Chart.yaml

@@ -14,7 +14,7 @@ type: application
 
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
-version: 2.3.0
+version: 2.3.1
 appVersion: "3.21.2"
 
 maintainers:

charts/saleor/templates/migration_job.yaml

@@ -23,8 +23,8 @@ spec:
       {{- end }}
       containers:
         - name: migrations
-          image: "{{ .Values.api.image.repository }}:{{ .Values.api.image.tag | default .Chart.AppVersion }}"
-          imagePullPolicy: {{ .Values.api.image.pullPolicy }}
+          image: "{{ .Values.global.image.repository }}:{{ .Values.global.image.tag | default .Chart.AppVersion }}"
+          imagePullPolicy: {{ .Values.global.image.pullPolicy }}
           command: ["python", "manage.py", "migrate", "--no-input"]
           env:
             # Core database configuration

charts/saleor/templates/saleor_deployment.yaml

@@ -49,8 +49,8 @@ spec:
         - name: {{ .Chart.Name }}-api
           securityContext:
             {{- toYaml .Values.api.securityContext | nindent 12 }}
-          image: "{{ .Values.api.image.repository }}:{{ .Values.api.image.tag | default .Chart.AppVersion }}"
-          imagePullPolicy: {{ .Values.api.image.pullPolicy }}
+          image: "{{ .Values.global.image.repository }}:{{ .Values.global.image.tag | default .Chart.AppVersion }}"
+          imagePullPolicy: {{ .Values.global.image.pullPolicy }}
           ports:
             - name: http
               containerPort: {{ .Values.api.service.port }}

charts/saleor/templates/worker_deployment.yaml

@@ -49,8 +49,8 @@ spec:
         - name: {{ .Chart.Name }}-worker
           securityContext:
             {{- toYaml .Values.securityContext | nindent 12 }}
-          image: "{{ .Values.worker.image.repository }}:{{ .Values.worker.image.tag | default .Chart.AppVersion }}"
-          imagePullPolicy: {{ .Values.worker.image.pullPolicy }}
+          image: "{{ .Values.global.image.repository }}:{{ .Values.global.image.tag | default .Chart.AppVersion }}"
+          imagePullPolicy: {{ .Values.global.image.pullPolicy }}
           command: ["celery"]
           args: ["--app=saleor.celeryconf:app", "worker", "-E", "--loglevel=info" ]
           livenessProbe:

charts/saleor/values.yaml

@@ -11,6 +11,14 @@ global:
   redisUrl: ""
   # -- External Celery Redis URL (if not using internal Redis). Use /0 for general usage and /1 for Celery for example
   celeryRedisUrl: ""
+  # -- Global image configuration for Saleor components
+  image:
+    # -- Global image repository for Saleor components
+    repository: ghcr.io/saleor/saleor
+    # -- Global image tag for Saleor components
+    tag: "3.21.2"
+    # -- Global image pull policy for Saleor components
+    pullPolicy: IfNotPresent
   # -- Database configuration
   database:
     # -- External primary database URL
@@ -56,11 +64,6 @@ api:
   enabled: true
   replicaCount: 2
 
-  image:
-    repository: ghcr.io/saleor/saleor
-    tag: "3.21.2"
-    pullPolicy: IfNotPresent
-
   extraEnv:
     # -- List of allowed hosts for Django. Must be exact hostnames, wildcards (*) are not supported.
     # For development, you can use ["*"] to allow all hosts (not recommended for production).
@@ -142,11 +145,6 @@ worker:
   enabled: true
   replicaCount: 1
 
-  image:
-    repository: ghcr.io/saleor/saleor
-    tag: "3.21.2"
-    pullPolicy: IfNotPresent
-  
   extraEnv:
 
   resources: