Merge pull request #3 from chainflip-io/chore/hook-annotations

.gitignore

@@ -1,2 +1,2 @@
-test.*.yaml
+test.*.*
 .idea

charts/node/Chart.yaml

@@ -10,6 +10,6 @@ maintainers:
 
 type: application
 
-version: 0.1.1
+version: 0.1.2
 
 appVersion: "0.8.7"

charts/node/templates/preinst-create-keys.yaml → charts/node/templates/hook-create-keys.yaml

@@ -9,6 +9,9 @@ metadata:
     "helm.sh/hook": post-install
     "helm.sh/hook-weight": "-5"
     "helm.sh/hook-delete-policy": hook-succeeded, hook-failed
+    {{- with .Values.hook.extraAnnotations }}
+    {{- toYaml . | nindent 4 }}
+    {{- end }}
 spec:
   template:
     metadata:

charts/node/tests/hook_test.yaml

@@ -0,0 +1,44 @@
+suite: hook tests
+templates:
+  - hook-create-keys.yaml
+  - hook-rbac.yaml
+tests:
+  - it: should template the create-keys hook if generateKeysOnStart is true
+    set:
+      common.keys.generateKeysOnStart: true
+    asserts:
+      - hasDocuments:
+          count: 1
+        template: hook-create-keys.yaml
+      - hasDocuments:
+          count: 3
+        template: hook-rbac.yaml
+      - isKind:
+          of: Job
+        template: hook-create-keys.yaml
+      - isKind:
+          of: ServiceAccount
+        template: hook-rbac.yaml
+        documentIndex: 0
+      - isKind:
+          of: Role
+        template: hook-rbac.yaml
+        documentIndex: 1
+      - isKind:
+          of: RoleBinding
+        template: hook-rbac.yaml
+        documentIndex: 2
+  - it: should create annotations on the job if set
+    set:
+      common.keys.generateKeysOnStart: true
+      hook.extraAnnotations:
+        foo: bar
+    asserts:
+      - hasDocuments:
+          count: 1
+        template: hook-create-keys.yaml
+      - equal:
+          path: metadata.annotations.foo
+          value: bar
+        documentIndex: 0
+        template: hook-create-keys.yaml

charts/node/values.yaml

@@ -253,5 +253,10 @@ cli:
     # -- chainflip-cli image pull policy
     pullPolicy: Always
 
+# -- hook configuration
+hook:
+  # -- add extra annotations to the create-keys hook
+  extraAnnotations: {}
+
 # -- create extra kubernetes manifests
 extraManifests: []