You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by el...@apache.org on 2025/04/28 03:40:43 UTC

(airflow) branch main updated: fix: replace break function in `pod-launcher-rolebinding template` (#49219)

This is an automated email from the ASF dual-hosted git repository.

eladkal pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new e836a410552 fix: replace break function in `pod-launcher-rolebinding template` (#49219)
e836a410552 is described below

commit e836a410552a0e279b5ed5fb1d2f1ccafee98206
Author: Guan Ming(Wesley) Chiu <10...@users.noreply.github.com>
AuthorDate: Mon Apr 28 11:40:36 2025 +0800

    fix: replace break function in `pod-launcher-rolebinding template` (#49219)
---
 chart/templates/rbac/pod-launcher-rolebinding.yaml | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/chart/templates/rbac/pod-launcher-rolebinding.yaml b/chart/templates/rbac/pod-launcher-rolebinding.yaml
index 2ff8f518a1d..adbee311d73 100644
--- a/chart/templates/rbac/pod-launcher-rolebinding.yaml
+++ b/chart/templates/rbac/pod-launcher-rolebinding.yaml
@@ -58,20 +58,22 @@ roleRef:
   name: {{ include "airflow.fullname" . }}-pod-launcher-role
   {{- end }}
 subjects:
+  {{- $schedulerAdded := false }}
   {{- range $executor := $executors }}
-  {{- if has $executor $schedulerLaunchExecutors }}
+  {{- if and (has $executor $schedulerLaunchExecutors) (not $schedulerAdded) }}
+  {{- $schedulerAdded = true }}
   - kind: ServiceAccount
     name: {{ include "scheduler.serviceAccountName" $ }}
     namespace: "{{ $.Release.Namespace }}"
-    {{- break -}}
   {{- end }}
   {{- end }}
+  {{- $workerAdded := false }}
   {{- range $executor := $executors }}
-  {{- if has $executor $workerLaunchExecutors }}
+  {{- if and (has $executor $workerLaunchExecutors) (not $workerAdded) }}
+  {{- $workerAdded = true }}
   - kind: ServiceAccount
     name: {{ include "worker.serviceAccountName" $ }}
     namespace: "{{ $.Release.Namespace }}"
-    {{- break -}}
   {{- end }}
   {{- end }}
 {{- end }}