Merge pull request #16 from paritytech/node-role

Allow setting the node role in the chart options
paritytech · Sep 14, 2021 · 6ae66c3 · 6ae66c3
2 parents 0b6c026 + c171606
commit 6ae66c3
Show file tree

Hide file tree

Showing 5 changed files with 18 additions and 3 deletions.
diff --git a/charts/node/Chart.yaml b/charts/node/Chart.yaml
@@ -2,5 +2,5 @@ apiVersion: v2
 name: node
 description: A Helm chart to deploy Substrate/Polkadot nodes
 type: application
-version: 0.11.0
+version: 0.12.0
 appVersion: "0.0.1"
diff --git a/charts/node/README.md b/charts/node/README.md
@@ -50,6 +50,7 @@ helm install kusama-node parity/node --set node.chainDataSnapshotUrl=https://ksm
 | `node.enableReadinessProbe`              | If true, enable the readiness probe check                                                                                 | `true`                         |
 | `node.dataVolumeSize`                    | The size of the chain data PersistentVolume                                                                               | `100Gi`                        |
 | `node.replica`                           | Number of replica in the node StatefulSet                                                                                 | `1`                            |
+| `node.role`                              | Set the role of the node: `full`, `authority` (for validators) or `light`                                                 | `full`                            |
 | `node.chainDataSnapshotUrl`              | Download and load chain data from a snapshot archive http URL                                                             | ``                             |
 | `node.chainDataSnapshotExtractionPath`   | The path at which the snapshot archive downloaded from a http URL will be extracted                                       | `/data/chains/${CHAIN_PATH}`   |
 | `node.chainDataSnapshotFormat`           | The snapshot archive format (`tar` or `7z`)                                                                               | `tar`                          |

diff --git a/charts/node/templates/_helpers.tpl b/charts/node/templates/_helpers.tpl
@@ -56,6 +56,7 @@ Service labels
 {{- define "chart.serviceLabels" -}}
 chain: {{ .Values.node.chain }}
 release: {{ .Release.Name }}
+role: {{ .Values.node.role }}
 {{- end }}
 
 {{/*

diff --git a/charts/node/templates/statefulset.yaml b/charts/node/templates/statefulset.yaml
@@ -134,16 +134,28 @@ spec:
                 --name=${POD_NAME} \
                 --base-path=/data/ \
                 --chain=${CHAIN} \
+                {{- if eq .Values.node.role "authority" }}
+                --validator \
+                {{- end }}
+                {{- if eq .Values.node.role "light" }}
+                --light \
+                {{- end }}
                 --listen-addr={{ .Values.node.perNodeServices.listenAddressBase }}${P2P_PORT} \
                 {{- if .Values.node.persistGeneratedNodeKey }}
-                --node-key-file /data/node-key" \
+                --node-key-file /data/node-key \
                 {{- end }}
-              {{- join " " .Values.node.flags | nindent 18 }}
+                {{- join " " .Values.node.flags | nindent 16 }}
           {{- else }}
           args:
             - --name=$(POD_NAME)
             - --base-path=/data/
             - --chain=$(CHAIN)
+            {{- if eq .Values.node.role "authority" }}
+            - --validator
+            {{- end }}
+            {{- if eq .Values.node.role "light" }}
+            - --light \
+            {{- end }}
             {{- if .Values.node.persistGeneratedNodeKey }}
             - --node-key-file=/data/node-key
             {{- end }}

diff --git a/charts/node/values.yaml b/charts/node/values.yaml
@@ -52,6 +52,7 @@ node:
   command: "polkadot"
   dataVolumeSize: 100Gi
   replicas: 1
+  role: full
   #chainDataSnapshotUrl: ""
   chainDataSnapshotExtractionPath: "/data/chains/${CHAIN_PATH}"
   chainDataSnapshotFormat: tar