fix(helm): replace deprecated hyperkube image in pre-delete hook (#214)

Signed-off-by: shubham <shubham.bajpai@mayadata.io>

charts/openebs/Chart.yaml

@@ -1,5 +1,5 @@
 apiVersion: v1
-version: 2.8.0
+version: 2.8.1
 name: openebs
 appVersion: 2.8.0
 description: Containerized Storage for Containers

charts/openebs/templates/cleanup-webhook.yaml

@@ -1,3 +1,9 @@
+# HELM first deletes RBAC, then it tries to delete other resources like SPC and PVC. 
+# We've got validating webhook on SPC and PVC.
+# But even that the policy of this webhook is Ignore, it fails because the ServiceAccount 
+# does not have permission to access resources like BDC anymore which are used for validation.
+# Therefore we first need to delete webhook so we can delete the rest of the deployments.
+{{- $kubeMinor := .Capabilities.KubeVersion.Minor | replace "+" "" }}
 apiVersion: batch/v1
 kind: Job
 metadata:
@@ -22,8 +28,9 @@ spec:
       {{- end }}
       containers:
         - name: kubectl
-          image: "{{ .Values.hyperkubeImage.repository }}:{{ .Values.hyperkubeImage.tag }}"
-          imagePullPolicy: "{{ .Values.hyperkubeImage.pullPolicy }}"
+          {{- /* bitnami maintains an image for all k8s versions */}}
+          {{- /* see: https://hub.docker.com/r/bitnami/kubectl */}}
+          image: {{ printf "%s/%s:%s.%s" "bitnami" "kubectl" .Capabilities.KubeVersion.Major $kubeMinor }}
           command:
           - /bin/sh
           - -c

charts/openebs/values.yaml

@@ -270,11 +270,3 @@ analytics:
   enabled: true
   # Specify in hours the duration after which a ping event needs to be sent.
   pingInterval: "24h"
-
-
-# To cleanup the validatingwebhook just before deletion of the charts
-# via a pre-delete webhook job
-hyperkubeImage:
-  repository: "k8s.gcr.io/hyperkube"
-  tag: "v1.12.1"
-  pullPolicy: "IfNotPresent"