From 6a055591aba89b7a95e366da787ae33f913df026 Mon Sep 17 00:00:00 2001
From: Nirmalya Sen <nsen@dremio.com>
Date: Thu, 21 Mar 2019 22:18:21 -0700
Subject: [PATCH] DX-15648: add imagePullPolicy attribute

Change-Id: I3eb55833c565ef0e7417369d3b8aa20d764f85c0
---
 charts/dremio/templates/dremio-coordinator.yaml | 1 +
 charts/dremio/templates/dremio-executor.yaml    | 2 ++
 charts/dremio/templates/dremio-master.yaml      | 3 +++
 charts/dremio/values.yaml                       | 7 ++++++-
 4 files changed, 12 insertions(+), 1 deletion(-)

diff --git a/charts/dremio/templates/dremio-coordinator.yaml b/charts/dremio/templates/dremio-coordinator.yaml
index 4724509f..bfede34c 100644
--- a/charts/dremio/templates/dremio-coordinator.yaml
+++ b/charts/dremio/templates/dremio-coordinator.yaml
@@ -27,6 +27,7 @@ spec:
       containers:
       - name: dremio-coordinator
         image: {{.Values.image}}
+        imagePullPolicy: IfNotPresent
         resources:
           requests:
             memory: {{.Values.coordinator.memory}}M
diff --git a/charts/dremio/templates/dremio-executor.yaml b/charts/dremio/templates/dremio-executor.yaml
index b24d3dec..8facdd0c 100644
--- a/charts/dremio/templates/dremio-executor.yaml
+++ b/charts/dremio/templates/dremio-executor.yaml
@@ -27,6 +27,7 @@ spec:
       containers:
       - name: dremio-executor
         image: {{.Values.image}}
+        imagePullPolicy: IfNotPresent
         resources:
           requests:
             memory: {{.Values.executor.memory}}M
@@ -61,6 +62,7 @@ spec:
       # since we're mounting a separate volume, reset permission to dremio uid/gid
       - name: chown-data-directory
         image: {{.Values.image}}
+        imagePullPolicy: IfNotPresent
         securityContext:
           runAsUser: 0
         volumeMounts:
diff --git a/charts/dremio/templates/dremio-master.yaml b/charts/dremio/templates/dremio-master.yaml
index f4f13aad..164e4f9e 100644
--- a/charts/dremio/templates/dremio-master.yaml
+++ b/charts/dremio/templates/dremio-master.yaml
@@ -30,6 +30,7 @@ spec:
       containers:
       - name: dremio-master-coordinator
         image: {{.Values.image}}
+        imagePullPolicy: IfNotPresent
         resources:
           requests:
             memory: {{.Values.coordinator.memory}}M
@@ -69,6 +70,7 @@ spec:
       # since we're mounting a separate volume, reset permission to dremio uid/gid
       - name: chown-data-directory
         image: {{.Values.image}}
+        imagePullPolicy: IfNotPresent
         securityContext:
           runAsUser: 0
         volumeMounts:
@@ -80,6 +82,7 @@ spec:
         - "/opt/dremio/data"
       - name: upgrade-task
         image: {{.Values.image}}
+        imagePullPolicy: IfNotPresent
         volumeMounts:
         - name: dremio-master-volume
           mountPath: /opt/dremio/data
diff --git a/charts/dremio/values.yaml b/charts/dremio/values.yaml
index 0aaf26b7..a63f96d6 100644
--- a/charts/dremio/values.yaml
+++ b/charts/dremio/values.yaml
@@ -1,4 +1,8 @@
-image: dremio/dremio-oss
+# The image used to build the Dremio cluster. It is recommended to update the
+# version tag to the version that you are using. This will ensure that all
+# the pods are using the same version of the software.
+image: dremio/dremio-oss:latest
+
 # Check out Dremio documentation for memory and cpu requirements for
 # the coordinators and the executors.
 # The value of memory should be in MB. CPU is in no of cores.
@@ -18,6 +22,7 @@ executor:
   cpu: 4
   count: 1
   volumeSize: 20Gi
+
 # If your Kubernetes cluster does not support LoadBalancer,
 # comment out the line below for the helm chart to succeed or add
 # the correct serviceType for your cluster.