Skip to content

Commit

Permalink
fix(helm): resource names (#131)
Browse files Browse the repository at this point in the history
* fix(helm): resource names

Replace .Release.Name with operator.fullname

* Update changelog.

* fix typo

* fix pr number
  • Loading branch information
razvan authored Jan 2, 2024
1 parent ae69bf7 commit 38c7672
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 5 deletions.
6 changes: 6 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,12 @@ All notable changes to this project will be documented in this file.

## [Unreleased]

### Fixed

- Replace "Release.Name" with "operator.fullname" in Helm resource names ([#131])

[#131]: https://github.com/stackabletech/listener-operator/pull/131

## [23.11.0] - 2023-11-24

### Added
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
apiVersion: apps/v1
kind: Deployment
metadata:
name: {{ .Release.Name }}-controller-deployment
name: {{ include "operator.fullname" . }}-deployment
labels:
{{- include "operator.labels" . | nindent 4 }}
spec:
Expand All @@ -24,7 +24,7 @@ spec:
imagePullSecrets:
{{- toYaml . | nindent 8 }}
{{- end }}
serviceAccountName: {{ .Release.Name }}-serviceaccount
serviceAccountName: {{ include "operator.fullname" . }}-serviceaccount
securityContext:
{{- toYaml .Values.podSecurityContext | nindent 8 }}
containers:
Expand Down
4 changes: 2 additions & 2 deletions deploy/helm/listener-operator/templates/node-daemonset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
apiVersion: apps/v1
kind: DaemonSet
metadata:
name: {{ .Release.Name }}-node-daemonset
name: {{ include "operator.fullname" . }}-node-daemonset
labels:
{{- include "operator.labels" . | nindent 4 }}
spec:
Expand All @@ -24,7 +24,7 @@ spec:
imagePullSecrets:
{{- toYaml . | nindent 8 }}
{{- end }}
serviceAccountName: {{ .Release.Name }}-serviceaccount
serviceAccountName: {{ include "operator.fullname" . }}-serviceaccount
securityContext:
{{- toYaml .Values.podSecurityContext | nindent 8 }}
containers:
Expand Down
2 changes: 1 addition & 1 deletion deploy/helm/listener-operator/templates/roles.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ volumes:
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
name: {{ .Release.Name }}-clusterrole
name: {{ include "operator.fullname" . }}-clusterrole
rules:
- apiGroups:
- ""
Expand Down

0 comments on commit 38c7672

Please sign in to comment.