remove image pull from node-init daemonset

selkies-project · Aug 20, 2020 · 24a8373 · 24a8373
1 parent e713e29
commit 24a8373
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 30 deletions.
diff --git a/examples/jupyter-notebook/bundle-manifests/patch-user-pod.yaml.tmpl b/examples/jupyter-notebook/bundle-manifests/patch-user-pod.yaml.tmpl
@@ -39,8 +39,8 @@ spec:
         app.broker/user: "{{.User}}"
     spec:
       nodeSelector:
-        app.broker/initialized: "true"
         app.broker/tier: {{ .NodeTier.NodeLabel }}
+        cloud.google.com/gke-nodepool: "{{ .NodeTier.NodeLabel }}"
       affinity:
         nodeAffinity:
           requiredDuringSchedulingIgnoredDuringExecution:

diff --git a/setup/manifests/base/node/node-init.yaml b/setup/manifests/base/node/node-init.yaml
@@ -86,39 +86,13 @@ spec:
           operator: "Exists"
       volumes:
         ###
-        # Local docker socket for pulling images
-        ###
-        - name: docker
-          hostPath:
-            path: /var/run/docker.sock
-            type: File
-        ###
         # Root filesystem from host
         ###
         - name: hostfs
           hostPath:
             path: /
       initContainers:
         ###
-        # Pull images
-        # This helps to speed up container startup with large images.
-        ###
-        - name: pull-images
-          image: gcr.io/cloud-solutions-images/kube-pod-broker-controller:latest
-          command: ["/usr/local/bin/image-puller"]
-          env:
-            - name: NAMESPACE
-              valueFrom:
-                fieldRef:
-                  fieldPath: metadata.namespace
-            - name: NODE_NAME
-              valueFrom:
-                fieldRef:
-                  fieldPath: spec.nodeName
-          volumeMounts:
-            - name: docker
-              mountPath: /var/run/docker.sock
-        ###
         # node init, set inotify limits, update node taint and label.
         ###
         - name: node-init
@@ -186,8 +160,7 @@ spec:
               chmod +x /tmp/init.sh
 
               while true; do
-                # Run init script in chroot
-                /tmp/init.sh
+                /tmp/init.sh | grep -qv "not labeled" || true
                 sleep 10
               done
           env:

diff --git a/setup/scripts/selkies_cli.sh b/setup/scripts/selkies_cli.sh
@@ -23,7 +23,7 @@ APP=${APP,,}
 
 [[ "${ACTION}" != "list" && -z "${APP}" ]] && echo "ERROR: missing app name for action: ${ACTION}" && exit 1
 
-ACCOUNT=$(gcloud config get-value account)
+ACCOUNT=${ACCOUNT:-$(gcloud config get-value account)}
 [[ -z "${ACCOUNT}" ]] && echo "ERROR: Failed to get gcloud account, did you run 'gcloud auth login'?" && exit 1
 
 case $ACTION in