Skip to content
This repository has been archived by the owner on Oct 22, 2021. It is now read-only.

Commit

Permalink
fix: rename apps-dns deployment
Browse files Browse the repository at this point in the history
Renaming apps-dns deployment to cf-apps-dns to get rid of the bad labels
and selectors have the desired effect.

Signed-off-by: Thulio Ferraz Assis <thulio.assis@suse.com>
  • Loading branch information
Thulio Ferraz Assis committed Sep 23, 2020
1 parent 5128be6 commit 8ac9670
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions chart/templates/apps_dns.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ data:
apiVersion: apps/v1
kind: Deployment
metadata:
name: apps-dns
name: cf-apps-dns
namespace: {{ .Release.Namespace | quote }}
labels:
app: apps-dns
Expand All @@ -87,16 +87,13 @@ spec:
app.kubernetes.io/instance: {{ .Release.Name | quote }}
app.kubernetes.io/managed-by: {{ .Release.Service | quote }}
app.kubernetes.io/name: {{ include "kubecf.fullname" . }}
helm.sh/chart: {{ include "kubecf.chart" . }}
template:
metadata:
labels:
app: apps-dns
app.kubernetes.io/instance: {{ .Release.Name | quote }}
app.kubernetes.io/managed-by: {{ .Release.Service | quote }}
app.kubernetes.io/name: {{ include "kubecf.fullname" . }}
app.kubernetes.io/version: {{ default .Chart.Version .Chart.AppVersion | quote }}
helm.sh/chart: {{ include "kubecf.chart" . }}
spec:
{{- if $.Values.sizing.apps_dns.affinity }}
affinity: {{ $.Values.sizing.apps_dns.affinity | toJson }}
Expand Down

0 comments on commit 8ac9670

Please sign in to comment.