fix: wrong template for broker extraEnv (#1134)

* fix: wrong template for broker extraEnv Signed-off-by: ericsyh <[email protected]> * apply to sn-platform chart Signed-off-by: ericsyh <[email protected]> * fix lint Signed-off-by: ericsyh <[email protected]> * update codeowner Signed-off-by: ericsyh <[email protected]> --------- Signed-off-by: ericsyh <[email protected]> (cherry picked from commit b285d71)
streamnative · Dec 21, 2023 · bd969a7 · bd969a7
1 parent 28a9320
commit bd969a7
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS
@@ -1 +1 @@
-*                       @streamnative/cloud @sijie
+*                       @streamnative/cloud @streamnative/cloud-data-plane
diff --git a/charts/sn-platform-slim/templates/broker/broker-cluster.yaml b/charts/sn-platform-slim/templates/broker/broker-cluster.yaml
@@ -105,7 +105,7 @@ spec:
     {{- end }}
     affinity:
       {{- include "pulsar.antiAffinityRules" (dict "Values" .Values "thisAffinity" .Values.broker.affinity "Component" .Values.broker.component "Release" .Release "Chart" .Chart) | indent 6 }}
-    {{- if or .Values.auth.authentication.enabled (and .Values.broker.offload.s3.enabled .Values.broker.offload.s3.secret) (and .Values.broker.offload.azureblob.enabled .Values.broker.offload.azureblob.secret) }}
+    {{- if or .Values.broker.extraEnv .Values.auth.authentication.enabled (and .Values.broker.offload.s3.enabled .Values.broker.offload.s3.secret) (and .Values.broker.offload.azureblob.enabled .Values.broker.offload.azureblob.secret) }}
     vars:
     {{- include "pulsar.authEnvironment" (dict "Values" .Values "Component" .Values.broker.component "Release" .Release "Chart" .Chart) | indent 4 }}
     {{- if and .Values.broker.offload.s3.enabled .Values.broker.offload.s3.secret }}

diff --git a/charts/sn-platform/templates/broker/broker-cluster.yaml b/charts/sn-platform/templates/broker/broker-cluster.yaml
@@ -105,7 +105,7 @@ spec:
     {{- end }}
     affinity:
       {{- include "pulsar.antiAffinityRules" (dict "Values" .Values "thisAffinity" .Values.broker.affinity "Component" .Values.broker.component "Release" .Release "Chart" .Chart) | indent 6 }}
-    {{- if or .Values.auth.authentication.enabled (and .Values.broker.offload.s3.enabled .Values.broker.offload.s3.secret) (and .Values.broker.offload.azureblob.enabled .Values.broker.offload.azureblob.secret) }}
+    {{- if or .Values.broker.extraEnv .Values.auth.authentication.enabled (and .Values.broker.offload.s3.enabled .Values.broker.offload.s3.secret) (and .Values.broker.offload.azureblob.enabled .Values.broker.offload.azureblob.secret) }}
     vars:
     {{- include "pulsar.authEnvironment" (dict "Values" .Values "Component" .Values.broker.component "Release" .Release "Chart" .Chart) | indent 4 }}
     {{- if and .Values.broker.offload.s3.enabled .Values.broker.offload.s3.secret }}
Original file line number	Diff line number	Diff line change
		@@ -1 +1 @@
		* @streamnative/cloud @sijie
		* @streamnative/cloud @streamnative/cloud-data-plane