Skip to content

Commit 35e8866

Browse files
authored
Merge pull request #23 from kubean-io/fix-helm-issue
fix helm issue
2 parents b17a57c + 8ca78d4 commit 35e8866

File tree

4 files changed

+14
-8
lines changed

4 files changed

+14
-8
lines changed

.github/workflows/release.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,4 +34,4 @@ jobs:
3434
make helm-chart-release
3535
else
3636
echo "Release already exists"
37-
fi
37+
fi

Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11

22
.PHONY: test
33
test:
4-
# yamllint -s .
4+
yamllint -s . || true
55
helm lint charts/ --strict
66

77
GITHUB_OWNER?=kubean-io

charts/templates/_helpers.tpl

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,11 +35,19 @@ Create chart name and version as used by the chart label.
3535
Common labels
3636
*/}}
3737
{{- define "kube-node-tuning.labels" -}}
38-
app.kubernetes.io/name: {{ include "kube-node-tuning.name" . }}
3938
helm.sh/chart: {{ include "kube-node-tuning.chart" . }}
40-
app.kubernetes.io/instance: {{ .Release.Name }}
4139
{{- if .Chart.AppVersion }}
4240
app.kubernetes.io/version: {{ .Chart.AppVersion | quote }}
4341
{{- end }}
4442
app.kubernetes.io/managed-by: {{ .Release.Service }}
43+
{{ include "kube-node-tuning.selectorLabels" . }}
44+
45+
{{- end -}}
46+
47+
{{/*
48+
Selector labels
49+
*/}}
50+
{{- define "kube-node-tuning.selectorLabels" -}}
51+
app.kubernetes.io/name: {{ include "kube-node-tuning.name" . }}
52+
app.kubernetes.io/instance: {{ .Release.Name }}
4553
{{- end -}}

charts/templates/daemonset.yaml

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,7 @@ spec:
2626
spec:
2727
initContainers:
2828
- name: setup-sysctl
29-
image: >
30-
{{ .Values.image.registry }}/{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}
29+
image: {{ .Values.image.registry }}/{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}
3130
imagePullPolicy: {{ .Values.image.pullPolicy }}
3231
command:
3332
- /bin/sh
@@ -43,8 +42,7 @@ spec:
4342
runAsUser: 0
4443
containers:
4544
- name: kube-node-tuning
46-
image: >
47-
{{ .Values.image.registry }}/{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}
45+
image: {{ .Values.image.registry }}/{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}
4846
imagePullPolicy: {{ .Values.image.pullPolicy }}
4947
command:
5048
- sleep

0 commit comments

Comments
 (0)