From 38a0f2781dd8f47df0863f46785714d1f731ec82 Mon Sep 17 00:00:00 2001
From: Nikolai Rodionov <allanger@badhouseplants.net>
Date: Wed, 16 Oct 2024 21:20:58 +0200
Subject: [PATCH] Fix SA in the roles chart

---
 charts/roles/templates/sa.yaml          | 2 +-
 values/badhouseplants/values.roles.yaml | 5 +++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/charts/roles/templates/sa.yaml b/charts/roles/templates/sa.yaml
index ce02cf0..fc554f3 100644
--- a/charts/roles/templates/sa.yaml
+++ b/charts/roles/templates/sa.yaml
@@ -1,5 +1,5 @@
 {{- if .Values.sa }}
-{{- range $sa := .Values.roles }}
+{{- range $sa := .Values.sa }}
 ---
 apiVersion: v1
 kind: ServiceAccount
diff --git a/values/badhouseplants/values.roles.yaml b/values/badhouseplants/values.roles.yaml
index 8d74a12..16dbaee 100644
--- a/values/badhouseplants/values.roles.yaml
+++ b/values/badhouseplants/values.roles.yaml
@@ -13,11 +13,12 @@ bindings:
     kind: ClusterRoleBinding
     subjects:
       - kind: ServiceAccount
-        name: pipelines
-        namespace: woodpecker-ci
+        namespace: pipelines
+        name: woodpecker-ci
     roleRef:
       kind: ClusterRole
       name: cluster-admin
       apiGroup: rbac.authorization.k8s.io
 sa:
   - name: woodpecker-ci
+    namespace: pipelines