Fix ace-installer chart (#542)

Signed-off-by: Tamal Saha <[email protected]> Signed-off-by: Rokibul Hasan <[email protected]> Co-authored-by: Rokibul Hasan <[email protected]>
appscode-cloud · Oct 25, 2024 · 8a51fab · 8a51fab
1 parent 9ed165f
commit 8a51fab
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/charts/ace-installer/templates/precheck/job.yaml b/charts/ace-installer/templates/precheck/job.yaml
@@ -6,11 +6,11 @@ metadata:
   name: {{ include "ace-installer.fullname" . }}-precheck
   namespace: {{ .Release.Namespace }}
   annotations:
-    "helm.sh/hook-weight": "0"
+    "helm.sh/hook-weight": "1"
     "helm.sh/hook": pre-install,pre-upgrade,pre-rollback
     "helm.sh/hook-delete-policy": before-hook-creation,hook-succeeded
 spec:
-  backoffLimit: 4
+  backoffLimit: 0
   manualSelector: true
   selector:
     matchLabels:

diff --git a/charts/ace-installer/templates/precheck/rbac.yaml b/charts/ace-installer/templates/precheck/rbac.yaml
@@ -12,15 +12,11 @@ metadata:
     "helm.sh/hook-delete-policy": before-hook-creation,hook-succeeded
 rules:
   - apiGroups: [""]
-    resources: ["configmaps"]
-    resourceNames: ["cluster-info"]
-    verbs: ["get"]
+    resources: ["configmaps", "namespaces", "secrets", "services"]
+    verbs: [ "get", "list", "watch" ]
   - apiGroups: ["apps"]
     resources: ["deployments"]
     verbs: ["get", "list", "watch"]
-  - apiGroups: [""]
-    resources: ["services"]
-    verbs: ["get", "list", "watch"]
   - apiGroups: ["storage.k8s.io"]
     resources: ["storageclasses"]
     verbs: ["get", "list", "watch"]