Skip to content

Commit 1471c04

Browse files
ixxeL2097Frederic Spiers
andauthored
feat(helm): add labels and additional rolebinding SA for certmanager (#90)
Co-authored-by: Frederic Spiers <[email protected]>
1 parent a316e53 commit 1471c04

File tree

2 files changed

+24
-0
lines changed

2 files changed

+24
-0
lines changed

helm/ggbridge/templates/rbac.yaml

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ apiVersion: v1
66
kind: ServiceAccount
77
metadata:
88
name: {{ printf "%s-issuer" $fullname }}
9+
namespace: {{ .Release.Namespace }}
910
labels:
1011
{{- include "ggbridge.labels" . | nindent 4 }}
1112
{{- if or .Values.commonAnnotations .Values.serviceAccount.annotations }}
@@ -18,6 +19,8 @@ kind: Role
1819
metadata:
1920
name: {{ printf "%s-issuer" $fullname }}
2021
namespace: {{ .Release.Namespace }}
22+
labels:
23+
{{- include "ggbridge.labels" . | nindent 4 }}
2124
rules:
2225
- apiGroups: ['']
2326
resources: ['serviceaccounts/token']
@@ -29,10 +32,17 @@ kind: RoleBinding
2932
metadata:
3033
name: {{ printf "%s-issuer" $fullname }}
3134
namespace: {{ .Release.Namespace }}
35+
labels:
36+
{{- include "ggbridge.labels" . | nindent 4 }}
3237
subjects:
3338
- kind: ServiceAccount
3439
name: {{ .Values.tls.certManager.serviceAccount }}
3540
namespace: {{ .Values.tls.certManager.namespace }}
41+
{{- if .Values.tls.certManager.issuer.spec.vault.auth.kubernetes.serviceAccountRef }}
42+
- kind: ServiceAccount
43+
name: {{ .Values.tls.certManager.issuer.spec.vault.auth.kubernetes.serviceAccountRef.name }}
44+
namespace: {{ .Release.Namespace }}
45+
{{- end }}
3646
roleRef:
3747
apiGroup: rbac.authorization.k8s.io
3848
kind: Role
@@ -42,6 +52,9 @@ apiVersion: v1
4252
kind: Secret
4353
metadata:
4454
name: {{ printf "%s-issuer-token" $fullname }}
55+
namespace: {{ .Release.Namespace }}
56+
labels:
57+
{{- include "ggbridge.labels" . | nindent 4 }}
4558
annotations:
4659
kubernetes.io/service-account.name: {{ printf "%s-issuer" $fullname }}
4760
type: kubernetes.io/service-account-token
@@ -50,6 +63,8 @@ apiVersion: rbac.authorization.k8s.io/v1
5063
kind: ClusterRoleBinding
5164
metadata:
5265
name: {{ printf "%s-issuer" $fullname }}-token-reviewer
66+
labels:
67+
{{- include "ggbridge.labels" . | nindent 4 }}
5368
roleRef:
5469
apiGroup: rbac.authorization.k8s.io
5570
kind: ClusterRole
@@ -66,6 +81,8 @@ kind: Role
6681
metadata:
6782
name: {{ printf "%s-cert-manager" $fullname }}
6883
namespace: {{ .Release.Namespace }}
84+
labels:
85+
{{- include "ggbridge.labels" . | nindent 4 }}
6986
rules:
7087
- apiGroups:
7188
- ''
@@ -99,6 +116,8 @@ kind: RoleBinding
99116
metadata:
100117
name: {{ printf "%s-cert-manager" $fullname }}
101118
namespace: {{ .Release.Namespace }}
119+
labels:
120+
{{- include "ggbridge.labels" . | nindent 4 }}
102121
roleRef:
103122
apiGroup: rbac.authorization.k8s.io
104123
kind: Role

helm/ggbridge/templates/serviceaccount.yaml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ apiVersion: v1
33
kind: ServiceAccount
44
metadata:
55
name: {{ include "ggbridge.serviceAccountName" . }}
6+
namespace: {{ .Release.Namespace }}
67
labels:
78
{{- include "ggbridge.labels" . | nindent 4 }}
89
{{- if or .Values.commonAnnotations .Values.serviceAccount.annotations }}
@@ -17,6 +18,7 @@ metadata:
1718
labels:
1819
{{- include "ggbridge.labels" . | nindent 4 }}
1920
name: {{ include "ggbridge.fullname" . }}
21+
namespace: {{ .Release.Namespace }}
2022
rules:
2123
- apiGroups:
2224
- ''
@@ -32,6 +34,9 @@ apiVersion: rbac.authorization.k8s.io/v1
3234
kind: RoleBinding
3335
metadata:
3436
name: {{ include "ggbridge.fullname" . }}
37+
namespace: {{ .Release.Namespace }}
38+
labels:
39+
{{- include "ggbridge.labels" . | nindent 4 }}
3540
roleRef:
3641
apiGroup: rbac.authorization.k8s.io
3742
kind: Role

0 commit comments

Comments
 (0)