Use default kubernetes imagePullPolicy (#587)

* Use kubernetes default imagePullPolicy. * Use 0-latest as dev version for helm charts * Add comments for pullPolicy and set as Optional value Signed-off-by: Dolpher Du <dolpher.du@intel.com>
opea-project · Nov 27, 2024 · 0f21681 · 0f21681
1 parent da0846e
commit 0f21681
Show file tree

Hide file tree

Showing 149 changed files with 394 additions and 325 deletions.
diff --git a/.github/workflows/manual-freeze-tag.yaml b/.github/workflows/manual-freeze-tag.yaml
@@ -7,17 +7,17 @@ on:
   workflow_dispatch:
     inputs:
       oldversion:
-        default: "1.1-dev"
+        default: "0-latest"
         description: "Old helm version to be replaced"
         required: true
         type: string
       newversion:
-        default: "1.1.0"
+        default: "1.2.0"
         description: "New helm version to replace"
         required: true
         type: string
       imageversion:
-        default: "1.1"
+        default: "1.2"
         description: "New image version to replace"
         required: true
         type: string

diff --git a/.github/workflows/manual-release-charts.yaml b/.github/workflows/manual-release-charts.yaml
@@ -7,7 +7,7 @@ on:
   workflow_dispatch:
     inputs:
       version:
-        default: "1.1.0"
+        default: "0-latest"
         description: "Release Version"
         required: true
         type: string

diff --git a/helm-charts/agentqna/Chart.yaml b/helm-charts/agentqna/Chart.yaml
@@ -7,37 +7,37 @@ description: The Helm chart to deploy AgentQnA
 type: application
 dependencies:
   - name: agent
-    version: 1.0.0
+    version: 0-latest
     alias: worker
     repository: "file://../common/agent"
   - name: agent
-    version: 1.0.0
+    version: 0-latest
     alias: supervisor
     repository: "file://../common/agent"
   - name: tgi
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/tgi"
     condition: tgi.enabled
   - name: tei
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/tei"
   - name: embedding-usvc
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/embedding-usvc"
   - name: teirerank
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/teirerank"
   - name: reranking-usvc
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/reranking-usvc"
   - name: redis-vector-db
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/redis-vector-db"
   - name: retriever-usvc
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/retriever-usvc"
   - name: data-prep
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/data-prep"
-version: 1.0.0
+version: 0-latest
 appVersion: "v1.0"
diff --git a/helm-charts/agentqna/templates/crag.yaml b/helm-charts/agentqna/templates/crag.yaml
@@ -40,7 +40,9 @@ spec:
           securityContext:
             {{- toYaml .Values.securityContext | nindent 12 }}
           image: "{{ .Values.crag.image.repository }}:{{ .Values.crag.image.tag | default .Chart.AppVersion }}"
+          {{- if .Values.crag.image.pullPolicy }}
           imagePullPolicy: {{ .Values.crag.image.pullPolicy }}
+          {{- end }}
           volumeMounts:
             - mountPath: /tmp
               name: tmp

diff --git a/helm-charts/agentqna/templates/docretriever.yaml b/helm-charts/agentqna/templates/docretriever.yaml
@@ -58,7 +58,9 @@ spec:
           securityContext:
             {{- toYaml .Values.securityContext | nindent 12 }}
           image: "{{ .Values.docretriever.image.repository }}:{{ .Values.docretriever.image.tag | default .Chart.AppVersion }}"
+          {{- if .Values.docretriever.image.pullPolicy }}
           imagePullPolicy: {{ .Values.docretriever.image.pullPolicy }}
+          {{- end }}
           volumeMounts:
             - mountPath: /tmp
               name: tmp

diff --git a/helm-charts/agentqna/values.yaml b/helm-charts/agentqna/values.yaml
@@ -32,7 +32,8 @@ horizontalPodAutoscaler:
 docretriever:
   image:
     repository: opea/doc-index-retriever
-    pullPolicy: IfNotPresent
+    # Uncomment the following line to set desired image pull policy if needed, as one of Always, IfNotPresent, Never.
+    # pullPolicy: ""
     # Overrides the image tag whose default is the chart appVersion.
     tag: "latest"
 
@@ -42,7 +43,8 @@ worker:
     port: 9095
   image:
     repository: opea/agent-langchain
-    pullPolicy: IfNotPresent
+    # Uncomment the following line to set desired image pull policy if needed, as one of Always, IfNotPresent, Never.
+    # pullPolicy: ""
     # Overrides the image tag whose default is the chart appVersion.
     tag: "latest"
   strategy: rag_agent_llama
@@ -63,7 +65,8 @@ supervisor:
     port: 9090
   image:
     repository: opea/agent-langchain
-    pullPolicy: IfNotPresent
+    # Uncomment the following line to set desired image pull policy if needed, as one of Always, IfNotPresent, Never.
+    # pullPolicy: ""
     # Overrides the image tag whose default is the chart appVersion.
     tag: "latest"
   strategy: react_llama
@@ -82,7 +85,8 @@ supervisor:
 crag:
   image:
     repository: aicrowd/kdd-cup-24-crag-mock-api
-    pullPolicy: IfNotPresent
+    # Uncomment the following line to set desired image pull policy if needed, as one of Always, IfNotPresent, Never.
+    # pullPolicy: ""
     # Overrides the image tag whose default is the chart appVersion.
     tag: "v0"
 

diff --git a/helm-charts/audioqna/Chart.yaml b/helm-charts/audioqna/Chart.yaml
@@ -7,27 +7,27 @@ description: The Helm chart to deploy AudioQnA
 type: application
 dependencies:
   - name: asr
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/asr"
   - name: whisper
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/whisper"
   - name: tts
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/tts"
   - name: speecht5
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/speecht5"
   - name: tgi
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/tgi"
   - name: llm-uservice
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/llm-uservice"
 # Uncomment the following to enable UI when the UI image is ready on DockerHub
 # - name: ui
-#  version: 1.0.0
+#  version: 0-latest
 #  repository: "file://../common/ui"
 #  alias: audioqna-ui
-version: 1.0.0
+version: 0-latest
 appVersion: "1.1"
diff --git a/helm-charts/audioqna/templates/deployment.yaml b/helm-charts/audioqna/templates/deployment.yaml
@@ -48,7 +48,9 @@ spec:
           securityContext:
             {{- toYaml .Values.securityContext | nindent 12 }}
           image: "{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}"
+          {{- if .Values.image.pullPolicy }}
           imagePullPolicy: {{ .Values.image.pullPolicy }}
+          {{- end }}
           volumeMounts:
             - mountPath: /tmp
               name: tmp

diff --git a/helm-charts/audioqna/templates/nginx-deployment.yaml b/helm-charts/audioqna/templates/nginx-deployment.yaml
@@ -67,7 +67,9 @@ spec:
     spec:
       containers:
       - image: nginx:1.27.1
+        {{- if .Values.image.pullPolicy }}
         imagePullPolicy: {{ .Values.image.pullPolicy }}
+        {{- end }}
         name: nginx
         volumeMounts:
         - mountPath: /etc/nginx/conf.d

diff --git a/helm-charts/audioqna/values.yaml b/helm-charts/audioqna/values.yaml
@@ -9,7 +9,8 @@ replicaCount: 1
 
 image:
   repository: opea/audioqna
-  pullPolicy: IfNotPresent
+  # Uncomment the following line to set desired image pull policy if needed, as one of Always, IfNotPresent, Never.
+  # pullPolicy: ""
   # Overrides the image tag whose default is the chart appVersion.
   tag: "latest"
 

diff --git a/helm-charts/chatqna/Chart.yaml b/helm-charts/chatqna/Chart.yaml
@@ -7,35 +7,35 @@ description: The Helm chart to deploy ChatQnA
 type: application
 dependencies:
   - name: tgi
-    version: 1.0.0
+    version: 0-latest
     alias: tgi-guardrails
     repository: "file://../common/tgi"
     condition: guardrails-usvc.enabled
   - name: guardrails-usvc
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/guardrails-usvc"
     condition: guardrails-usvc.enabled
   - name: tgi
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/tgi"
   - name: tei
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/tei"
   - name: teirerank
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/teirerank"
   - name: redis-vector-db
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/redis-vector-db"
   - name: retriever-usvc
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/retriever-usvc"
   - name: data-prep
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/data-prep"
   - name: ui
     alias: chatqna-ui
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/ui"
-version: 1.0.0
+version: 0-latest
 appVersion: "v1.0"
diff --git a/helm-charts/chatqna/templates/deployment.yaml b/helm-charts/chatqna/templates/deployment.yaml
@@ -62,7 +62,9 @@ spec:
           securityContext:
             {{- toYaml .Values.securityContext | nindent 12 }}
           image: "{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}"
+          {{- if .Values.image.pullPolicy }}
           imagePullPolicy: {{ .Values.image.pullPolicy }}
+          {{- end }}
           volumeMounts:
             - mountPath: /tmp
               name: tmp

diff --git a/helm-charts/chatqna/templates/nginx-deployment.yaml b/helm-charts/chatqna/templates/nginx-deployment.yaml
@@ -89,7 +89,9 @@ spec:
     spec:
       containers:
       - image: nginx:1.27.1
+        {{- if .Values.image.pullPolicy }}
         imagePullPolicy: {{ .Values.image.pullPolicy }}
+        {{- end }}
         name: nginx
         volumeMounts:
         - mountPath: /etc/nginx/conf.d

diff --git a/helm-charts/chatqna/values.yaml b/helm-charts/chatqna/values.yaml
@@ -9,7 +9,8 @@ replicaCount: 1
 
 image:
   repository: opea/chatqna
-  pullPolicy: IfNotPresent
+  # Uncomment the following line to set desired image pull policy if needed, as one of Always, IfNotPresent, Never.
+  # pullPolicy: ""
   # Overrides the image tag whose default is the chart appVersion.
   tag: "latest"
 

diff --git a/helm-charts/codegen/Chart.yaml b/helm-charts/codegen/Chart.yaml
@@ -7,14 +7,14 @@ description: The Helm chart to deploy CodeGen
 type: application
 dependencies:
   - name: tgi
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/tgi"
   - name: llm-uservice
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/llm-uservice"
   - name: ui
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/ui"
     alias: codegen-ui
-version: 1.0.0
+version: 0-latest
 appVersion: "v1.0"
diff --git a/helm-charts/codegen/templates/deployment.yaml b/helm-charts/codegen/templates/deployment.yaml
@@ -40,7 +40,9 @@ spec:
           securityContext:
             {{- toYaml .Values.securityContext | nindent 12 }}
           image: "{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}"
+          {{- if .Values.image.pullPolicy }}
           imagePullPolicy: {{ .Values.image.pullPolicy }}
+          {{- end }}
           volumeMounts:
             - mountPath: /tmp
               name: tmp

diff --git a/helm-charts/codegen/templates/nginx-deployment.yaml b/helm-charts/codegen/templates/nginx-deployment.yaml
@@ -65,7 +65,9 @@ spec:
     spec:
       containers:
       - image: nginx:1.27.1
+        {{- if .Values.image.pullPolicy }}
         imagePullPolicy: {{ .Values.image.pullPolicy }}
+        {{- end }}
         name: nginx
         volumeMounts:
         - mountPath: /etc/nginx/conf.d

diff --git a/helm-charts/codegen/values.yaml b/helm-charts/codegen/values.yaml
@@ -9,7 +9,8 @@ replicaCount: 1
 
 image:
   repository: opea/codegen
-  pullPolicy: IfNotPresent
+  # Uncomment the following line to set desired image pull policy if needed, as one of Always, IfNotPresent, Never.
+  # pullPolicy: ""
   # Overrides the image tag whose default is the chart appVersion.
   tag: "latest"
 

diff --git a/helm-charts/codetrans/Chart.yaml b/helm-charts/codetrans/Chart.yaml
@@ -7,14 +7,14 @@ description: The Helm chart to deploy CodeTrans
 type: application
 dependencies:
   - name: tgi
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/tgi"
   - name: llm-uservice
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/llm-uservice"
   - name: ui
-    version: 1.0.0
+    version: 0-latest
     repository: "file://../common/ui"
     alias: codetrans-ui
-version: 1.0.0
+version: 0-latest
 appVersion: "v1.0"
diff --git a/helm-charts/codetrans/templates/deployment.yaml b/helm-charts/codetrans/templates/deployment.yaml
@@ -44,7 +44,9 @@ spec:
           securityContext:
             {{- toYaml .Values.securityContext | nindent 12 }}
           image: "{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}"
+          {{- if .Values.image.pullPolicy }}
           imagePullPolicy: {{ .Values.image.pullPolicy }}
+          {{- end }}
           volumeMounts:
             - mountPath: /tmp
               name: tmp

diff --git a/helm-charts/codetrans/templates/nginx-deployment.yaml b/helm-charts/codetrans/templates/nginx-deployment.yaml
@@ -65,7 +65,9 @@ spec:
     spec:
       containers:
       - image: nginx:1.27.1
+        {{- if .Values.image.pullPolicy }}
         imagePullPolicy: {{ .Values.image.pullPolicy }}
+        {{- end }}
         name: nginx
         volumeMounts:
         - mountPath: /etc/nginx/conf.d

diff --git a/helm-charts/codetrans/values.yaml b/helm-charts/codetrans/values.yaml
@@ -10,7 +10,8 @@ LLM_SERVICE_HOST_IP: ""
 
 image:
   repository: opea/codetrans
-  pullPolicy: IfNotPresent
+  # Uncomment the following line to set desired image pull policy if needed, as one of Always, IfNotPresent, Never.
+  # pullPolicy: ""
   # Overrides the image tag whose default is the chart appVersion.
   tag: "latest"
 

diff --git a/helm-charts/common/agent/Chart.yaml b/helm-charts/common/agent/Chart.yaml
@@ -5,15 +5,15 @@ apiVersion: v2
 name: agent
 description: The Helm chart for deploying agent microservice
 type: application
-version: 1.0.0
+version: 0-latest
 # The llm microservice server version
 appVersion: "v1.0"
 dependencies:
   - name: tgi
-    version: 1.0.0
+    version: 0-latest
     repository: file://../tgi
     condition: tgi.enabled
   - name: vllm
-    version: 1.0.0
+    version: 0-latest
     repository: file://../vllm
     condition: vllm.enabled