feat: support rackAwareTopologyLabels field on bk (#1196)

* feat: support rackAwareTopologyLabels field on bk Signed-off-by: ericsyh <[email protected]> * support on the rackAwareTopologyLabels on sn-plaform chart Signed-off-by: ericsyh <[email protected]> * fix lint Signed-off-by: ericsyh <[email protected]> --------- Signed-off-by: ericsyh <[email protected]> (cherry picked from commit d5acec5)
streamnative · Dec 20, 2024 · 704aaf2 · 704aaf2
1 parent 4d45067
commit 704aaf2
Show file tree

Hide file tree

Showing 4 changed files with 16 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
@@ -168,6 +168,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
@@ -844,6 +844,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:

diff --git a/charts/sn-platform/templates/bookkeeper/bookkeeper-cluster.yaml b/charts/sn-platform/templates/bookkeeper/bookkeeper-cluster.yaml
@@ -168,6 +168,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/values.yaml b/charts/sn-platform/values.yaml
@@ -920,6 +920,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: