feat: support rackAwareTopologyLabels field on bk

Signed-off-by: ericsyh <[email protected]>
streamnative · Dec 2, 2024 · c439190 · c439190
1 parent da726e7
commit c439190
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/charts/sn-platform-slim/templates/bookkeeper/bookkeeper-cluster.yaml b/charts/sn-platform-slim/templates/bookkeeper/bookkeeper-cluster.yaml
@@ -151,6 +151,10 @@ spec:
     reclaimPolicy: {{ .Values.bookkeeper.volumes.reclaimPolicy | default "Delete" }}
   {{- end }}
   config:
+    {{- with .Values.bookkeeper.rackAwareTopologyLabels }}
+    rackAwareTopologyLabels:
+    {{- toYaml . | nindent 4 }}
+    {{- end }}
     custom:
 {{- with .Values.bookkeeper.custom }}
 {{ toYaml . | indent 6 }}

diff --git a/charts/sn-platform-slim/values.yaml b/charts/sn-platform-slim/values.yaml
@@ -826,6 +826,10 @@ bookkeeper:
     # preferredDuringSchedulingIgnoredDuringExecution - scheduler will try to enforce but not guranentee
     type: preferredDuringSchedulingIgnoredDuringExecution
   topologySpreadConstraints: []
+  # For BookKeeperCluster, the field rackAwareTopologyLabels should be configured as nodes topology labels the value of which will be set as the rack name. 
+  # When there are multiple labels configured, all their values will be joined by "/". 
+  # The operator will propagate the label value from nodes to pods and ensure the pod container starts with the label values as environment variables automatically.
+  rackAwareTopologyLabels: []
   labels: {}
   annotations: {}
   securityContext: