Merge pull request #7 from chainflip-io/fix/cli

chainflip-io · Aug 21, 2023 · c497cbd · c497cbd
2 parents 9577dc0 + 5a78ddc
commit c497cbd
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/charts/node/Chart.yaml b/charts/node/Chart.yaml
@@ -10,6 +10,6 @@ maintainers:
 
 type: application
 
-version: 0.1.5
+version: 0.1.6
 
 appVersion: "0.8.7"
diff --git a/charts/node/templates/cli-deployment.yaml b/charts/node/templates/cli-deployment.yaml
@@ -1,4 +1,4 @@
-{{- if .Values.cli.enabled }}
+{{- if and (eq .Values.common.role "validator") .Values.cli.enabled }}
 apiVersion: apps/v1
 kind: Deployment
 metadata:

diff --git a/charts/node/tests/is_archive_test.yaml b/charts/node/tests/is_archive_test.yaml
@@ -2,6 +2,7 @@ suite: tests for if role is archive
 templates:
   - node-statefulset.yaml
   - engine-statefulset.yaml
+  - cli-deployment.yaml
 values:
   - values/happy.values.yaml
   - values/archive.values.yaml
@@ -20,4 +21,9 @@ tests:
     asserts:
       - hasDocuments:
           count: 0
-        template: engine-statefulset.yaml
+        template: engine-statefulset.yaml
+  - it: should not create the cli template
+    asserts:
+      - hasDocuments:
+          count: 0
+        template: cli-deployment.yaml