Skip to content

Commit aa4980e

Browse files
authored
Added missed imagePullPolicy for all CVAT deployments (cvat-ai#5504)
1 parent 321ba4f commit aa4980e

File tree

11 files changed

+14
-8
lines changed

11 files changed

+14
-8
lines changed

.github/workflows/helm.yml

-1
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,6 @@ jobs:
4545
- name: Deploy to minikube
4646
run: |
4747
printf "traefik:\n service:\n externalIPs:\n - $(minikube ip)\n" >> tests/values.test.yaml
48-
find cvat/apps/iam/rules -name "*.rego" -and ! -name '*test*' -exec basename {} \; | tar -czf helm-chart/rules.tar.gz -C cvat/apps/iam/rules/ -T -
4948
cd helm-chart
5049
helm dependency update
5150
cd ..

helm-chart/Chart.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ type: application
1515
# This is the chart version. This version number should be incremented each time you make changes
1616
# to the chart and its templates, including the app version.
1717
# Versions are expected to follow Semantic Versioning (https://semver.org/)
18-
version: 0.5.1
18+
version: 0.5.2
1919

2020
# This is the version number of the application being deployed. This version number should be
2121
# incremented each time you make changes to the application. Versions are not expected to

helm-chart/templates/cvat_backend/cvat_worker_webhooks/deployment.yml

+1
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ spec:
4646
containers:
4747
- name: cvat-app-backend-worker-webhooks-container
4848
image: {{ .Values.cvat.backend.image }}:{{ .Values.cvat.backend.tag }}
49+
imagePullPolicy: {{ .Values.cvat.backend.imagePullPolicy }}
4950
{{- with .Values.cvat.backend.worker.webhooks.resources }}
5051
resources:
5152
{{- toYaml . | nindent 12 }}

helm-chart/templates/cvat_backend/server/deployment.yml

+1
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ spec:
4646
containers:
4747
- name: cvat-app-backend-server-container
4848
image: {{ .Values.cvat.backend.image }}:{{ .Values.cvat.backend.tag }}
49+
imagePullPolicy: {{ .Values.cvat.backend.imagePullPolicy }}
4950
{{- with .Values.cvat.backend.server.resources }}
5051
resources:
5152
{{- toYaml . | nindent 12 }}

helm-chart/templates/cvat_backend/utils/deployment.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ spec:
4646
containers:
4747
- name: cvat-app-backend-utils-container
4848
image: {{ .Values.cvat.backend.image }}:{{ .Values.cvat.backend.tag }}
49-
imagePullPolicy: {{ .Values.cvat.backend.imagepullpolicy }}
49+
imagePullPolicy: {{ .Values.cvat.backend.imagePullPolicy }}
5050
{{- with .Values.cvat.backend.utils.resources }}
5151
resources:
5252
{{- toYaml . | nindent 12 }}

helm-chart/templates/cvat_backend/worker_default/deployment.yml

+1
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ spec:
4646
containers:
4747
- name: cvat-app-backend-worker-default-container
4848
image: {{ .Values.cvat.backend.image }}:{{ .Values.cvat.backend.tag }}
49+
imagePullPolicy: {{ .Values.cvat.backend.imagePullPolicy }}
4950
{{- with .Values.cvat.backend.worker.default.resources }}
5051
resources:
5152
{{- toYaml . | nindent 12 }}

helm-chart/templates/cvat_backend/worker_low/deployment.yml

+1
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ spec:
4646
containers:
4747
- name: cvat-app-backend-worker-low-container
4848
image: {{ .Values.cvat.backend.image }}:{{ .Values.cvat.backend.tag }}
49+
imagePullPolicy: {{ .Values.cvat.backend.imagePullPolicy }}
4950
{{- with .Values.cvat.backend.worker.low.resources }}
5051
resources:
5152
{{- toYaml . | nindent 12 }}

helm-chart/templates/cvat_frontend/deployment.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ spec:
3333
containers:
3434
- name: cvat-frontend-app-container
3535
image: {{ .Values.cvat.frontend.image }}:{{ .Values.cvat.frontend.tag }}
36-
imagePullPolicy: {{ .Values.cvat.frontend.imagepullpolicy }}
36+
imagePullPolicy: {{ .Values.cvat.frontend.imagePullPolicy }}
3737
{{- with .Values.cvat.frontend.resources }}
3838
resources:
3939
{{- toYaml . | nindent 12 }}

helm-chart/templates/cvat_opa/deployment.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ spec:
3333
containers:
3434
- name: cvat-opa-app-container
3535
image: {{ .Values.cvat.opa.image }}:{{ .Values.cvat.opa.tag }}
36-
imagePullPolicy: {{ .Values.cvat.opa.imagepullpolicy }}
36+
imagePullPolicy: {{ .Values.cvat.opa.imagePullPolicy }}
3737
args:
3838
- run
3939
- --server

helm-chart/values.yaml

+3-3
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ cvat:
7575
replicas: 1
7676
image: cvat/server
7777
tag: dev
78-
imagepullploicy: IfNotPresent
78+
imagePullPolicy: Always
7979
permissionFix:
8080
enabled: true
8181
service:
@@ -95,7 +95,7 @@ cvat:
9595
replicas: 1
9696
image: cvat/ui
9797
tag: dev
98-
imagepullploicy: IfNotPresent
98+
imagePullPolicy: Always
9999
labels: {}
100100
# test: test
101101
annotations: {}
@@ -138,7 +138,7 @@ cvat:
138138
replicas: 1
139139
image: openpolicyagent/opa
140140
tag: 0.34.2-rootless
141-
imagepullploicy: IfNotPresent
141+
imagePullPolicy: IfNotPresent
142142
labels: {}
143143
# test: test
144144
annotations: {}

tests/values.test.yaml

+3
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,9 @@ cvat:
1111
- mountPath: /home/django/share
1212
name: cvat-backend-data
1313
subPath: share
14+
# Images are already present in the node
15+
imagePullPolicy: Never
16+
1417
traefik:
1518
service:
1619
externalIPs:

0 commit comments

Comments
 (0)