Image pull policy

agent/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "agent"
-version = "0.1.12"
+version = "0.1.13"
 authors = ["Kate Goldenring <[email protected]>", "<[email protected]>"]
 edition = "2018"
 

controller/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "controller"
-version = "0.1.12"
+version = "0.1.13"
 authors = ["<[email protected]>"]
 edition = "2018"
 

deployment/helm/Chart.yaml

@@ -15,9 +15,9 @@ 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.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 0.1.12
+version: 0.1.13
 
 # This is the version number of the application being deployed. This version number should be
 # incremented each time you make changes to the application. Versions are not expected to
 # follow Semantic Versioning. They should reflect the version the application is using.
-appVersion: 0.1.12
+appVersion: 0.1.13

deployment/helm/templates/agent.yaml

@@ -36,7 +36,9 @@ spec:
         image: {{ printf "%s:%s" .Values.agent.image.repository (default (printf "v%s" .Chart.AppVersion) .Values.agent.image.tag) | quote }}
         {{- end }}
         {{- end }}
-        imagePullPolicy: {{ .Values.agent.image.pullPolicy }}
+        {{- with .Values.agent.image.pullPolicy }}
+        imagePullPolicy: {{ . }}
+        {{- end }}
         {{- if .Values.agent.securityContext }}
         securityContext:
         {{- toYaml .Values.agent.securityContext | nindent 10 }}

deployment/helm/templates/controller.yaml

@@ -31,7 +31,9 @@ spec:
         image: {{ printf "%s:%s" .Values.controller.image.repository (default (printf "v%s" .Chart.AppVersion) .Values.controller.image.tag) | quote }}
         {{- end }}
         {{- end }}
-        imagePullPolicy: {{ .Values.controller.image.pullPolicy }}
+        {{- with .Values.controller.image.pullPolicy }}
+        imagePullPolicy: {{ . }}
+        {{- end }}
         {{- if .Values.prometheus.enabled }}
         ports:
           - name: {{ .Values.prometheus.portName | quote }}

deployment/helm/templates/debug-echo.yaml

@@ -17,7 +17,9 @@ spec:
     containers:
     - name: {{ .Values.debugEcho.name }}-broker
       image: {{ .Values.debugEcho.brokerPod.image | quote }}
-      imagePullPolicy: {{ .Values.debugEcho.pullPolicy }}
+      {{- with .Values.debugEcho.pullPolicy }}
+      imagePullPolicy: {{ . }}
+      {{- end }}
       resources:
         limits:
           {{`"{{PLACEHOLDER}}"`}} : "1"

deployment/helm/templates/onvif.yaml

@@ -37,7 +37,9 @@ spec:
     containers:
     - name: {{ .Values.onvif.name }}-broker
       image: {{ .Values.onvif.brokerPod.image.repository | quote }}
-      imagePullPolicy: {{ .Values.onvif.brokerPod.image.pullPolicy }}
+      {{- with .Values.onvif.brokerPod.image.pullPolicy }}
+      imagePullPolicy: {{ . }}
+      {{- end }}
       {{- if .Values.onvif.brokerPod.env }}
       env:
         {{- range $key, $val := .Values.onvif.brokerPod.env }}

deployment/helm/templates/opcua.yaml

@@ -24,7 +24,9 @@ spec:
     containers:
     - name: {{ .Values.opcua.name }}-broker
       image: {{ .Values.opcua.brokerPod.image.repository | quote }}
-      imagePullPolicy: {{ .Values.opcua.brokerPod.image.pullPolicy }}
+      {{- with .Values.opcua.brokerPod.image.pullPolicy }}
+      imagePullPolicy: {{ . }}
+      {{- end }}
       {{- if .Values.opcua.brokerPod.env }}
       env:
         {{- range $key, $val := .Values.opcua.brokerPod.env }}

deployment/helm/templates/udev.yaml

@@ -14,7 +14,9 @@ spec:
     containers:
     - name: {{ .Values.udev.name }}-broker
       image: {{ .Values.udev.brokerPod.image.repository | quote }}
-      imagePullPolicy: {{ .Values.udev.brokerPod.image.pullPolicy }}
+      {{- with .Values.udev.brokerPod.image.pullPolicy }}
+      imagePullPolicy: {{ . }}
+      {{- end }}
       {{- if .Values.udev.brokerPod.env }}
       env:
         {{- range $key, $val := .Values.udev.brokerPod.env }}

deployment/helm/values.yaml

@@ -46,7 +46,7 @@ controller:
     # controller.yaml will default to v(AppVersion)-dev
     tag:
     # pullPolicy is the Akri Controller pull policy
-    pullPolicy: Always
+    pullPolicy: ""
   # onlyOnControlPlane dictates whether the Akri Controller will only run on nodes with 
   # the label with (key, value) of ("node-role.kubernetes.io/master", "")
   onlyOnControlPlane: false
@@ -69,7 +69,7 @@ agent:
     # agent.yaml will default to v(AppVersion)-dev
     tag:
     # pullPolicy is the Akri Agent pull policy
-    pullPolicy: Always
+    pullPolicy: ""
   securityContext: {}
   host:
     # kubeletDevicePlugins is the location of the kubelet device-plugin sockets
@@ -109,7 +109,7 @@ debugEcho:
     # container used by debugEcho
     image: nginx:latest
     # pullPolicy is the debugEcho pull policy
-    pullPolicy: Always
+    pullPolicy: ""
   instanceService:
     # name is the description of the instance service
     name: akri-debug-echo-foo-instance-service
@@ -160,7 +160,7 @@ onvif:
       # repository is the container reference
       repository:
       # pullPolicy is the Akri onvif broker pull policy
-      pullPolicy: Always
+      pullPolicy: ""
   # createInstanceServices is specified if a service should automatically be
   # created for each broker pod
   createInstanceServices: true
@@ -225,7 +225,7 @@ opcua:
       # repository is the Akri OPC UA broker container reference
       repository: 
       # pullPolicy is the Akri OPC UA broker pull policy
-      pullPolicy: Always
+      pullPolicy: ""
   # createInstanceServices is specified if a service should automatically be
   # created for each broker pod
   createInstanceServices: true
@@ -274,7 +274,7 @@ udev:
     image:
       # repository is the container reference
       repository:
-      pullPolicy: Always
+      pullPolicy: ""
     securityContext: {}
   # createInstanceServices is specified if a service should automatically be
   # created for each broker pod

docs/customizing-akri-installation.md

@@ -97,7 +97,6 @@ spec:
     containers:
     - name: akri-onvif-video-broker
       image: "ghcr.io/deislabs/akri/onvif-video-broker:latest-dev"
-      imagePullPolicy: Always
       resources:
         limits:
           "{{PLACEHOLDER}}" : "1"
@@ -114,7 +113,6 @@ spec:
     containers:
     - name: akri-onvif-video-broker
       image: "ghcr.io/deislabs/akri/onvif-video-broker:latest-dev"
-      imagePullPolicy: Always
       resources:
         requests:
           memory: 30Mi

docs/proposals/broker-deployment-strategies.md

@@ -42,7 +42,6 @@ spec:
     containers:
     - name: akri-onvif-video-broker
       image: "ghcr.io/..."
-      imagePullPolicy: Always
       resources:
         limits:
           "{{PLACEHOLDER}}" : "1"

samples/brokers/udev-video-broker/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "udev-video-broker"
-version = "0.1.12"
+version = "0.1.13"
 authors = ["Kate Goldenring <[email protected]>", "<[email protected]>"]
 edition = "2018"
 

shared/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "akri-shared"
-version = "0.1.12"
+version = "0.1.13"
 authors = ["<[email protected]>"]
 edition = "2018"
 

test/yaml/akri-onvif-video.yaml

@@ -23,7 +23,6 @@ spec:
     containers:
     - name: akri-onvif-video-broker
       image: "ghcr.io/deislabs/akri/onvif-video-broker:latest-dev"
-      imagePullPolicy: Always
       resources:
         limits:
           "{{PLACEHOLDER}}" : "1"

test/yaml/akri-opcua.yaml

@@ -16,7 +16,6 @@ spec:
     containers:
     - name: akri-opcua-broker
       image: "ghcr.io/deislabs/akri/opcua-monitoring-broker:latest-dev"
-      imagePullPolicy: Always
       env:
         - name: IDENTIFIER
           value: 

test/yaml/akri-udev-video.yaml

@@ -11,7 +11,6 @@ spec:
     containers:
     - name: akri-udev-video-broker
       image: "ghcr.io/deislabs/akri/udev-video-broker:latest-dev"
-      imagePullPolicy: Always
       env: 
         - name: FORMAT # default: MJPG
           value: "YUYV"

version.txt

@@ -1 +1 @@
-0.1.12
+0.1.13