Skip to content

Commit

Permalink
Replaces helper based serviceAccount reference with explicit value (#49)
Browse files Browse the repository at this point in the history
  • Loading branch information
rwc authored Jul 8, 2024
1 parent 4130ada commit af29e3e
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 6 deletions.
2 changes: 1 addition & 1 deletion charts/databricks-kube-operator/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
appVersion: 0.7.0
name: databricks-kube-operator
description: A kube-rs operator for managing Databricks API resources
version: 0.8.1
version: 0.8.2

home: https://github.com/mach-kernel/databricks-kube-operator
sources:
Expand Down
5 changes: 2 additions & 3 deletions charts/databricks-kube-operator/templates/rbac.yaml
Original file line number Diff line number Diff line change
@@ -1,10 +1,9 @@
{{- $svcAccount := include "databricks-kube-operator.serviceAccountName" . }}
{{- if .Values.serviceAccount.create }}
---
apiVersion: v1
kind: ServiceAccount
metadata:
name: "{{ $svcAccount }}"
name: {{ .Values.serviceAccount.name }}
namespace: {{ .Release.Namespace }}
{{- end }}
---
Expand Down Expand Up @@ -45,7 +44,7 @@ metadata:
subjects:
- namespace: {{ .Release.Namespace }}
kind: ServiceAccount
name: "{{ $svcAccount }}"
name: {{ .Values.serviceAccount.name }}
roleRef:
apiGroup: rbac.authorization.k8s.io
kind: ClusterRole
Expand Down
3 changes: 1 addition & 2 deletions charts/databricks-kube-operator/templates/sts.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
{{- $svcAccount := include "databricks-kube-operator.serviceAccountName" . }}
---
apiVersion: apps/v1
kind: StatefulSet
Expand All @@ -21,7 +20,7 @@ spec:
labels:
app: {{ template "databricks-kube-operator.name" . }}
spec:
serviceAccountName: "{{ $svcAccount }}"
serviceAccountName: {{ .Values.serviceAccount.name }}
terminationGracePeriodSeconds: 10
containers:
- name: dko
Expand Down

0 comments on commit af29e3e

Please sign in to comment.