Skip to content

Commit

Permalink
fix: dont use Chart appVersion in service chart
Browse files Browse the repository at this point in the history
  • Loading branch information
danielpanzella committed Oct 23, 2024
1 parent 03ef106 commit 40b4d37
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 6 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/pr-release.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -60,9 +60,8 @@ jobs:
shell: bash

- name: Run chart-releaser
uses: helm/chart-releaser-action@v1.6.0
uses: helm/chart-releaser-action@v1.5.0
with:
skip_packaging: true
mark_as_latest: false
env:
CR_TOKEN: "${{ secrets.GITHUB_TOKEN }}"
4 changes: 2 additions & 2 deletions charts/wandb-service/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,8 @@ Common labels
{{- define "wandb-service.labels" -}}
helm.sh/chart: {{ include "wandb-service.chart" . }}
{{ include "wandb-service.selectorLabels" . }}
{{- if .Chart.AppVersion }}
app.kubernetes.io/version: {{ .Chart.AppVersion | quote }}
{{- if .Values.image.tag }}
app.kubernetes.io/version: {{ .Values.image.tag | quote }}
{{- end }}
app.kubernetes.io/managed-by: {{ .Release.Service }}
{{- end }}
Expand Down
2 changes: 1 addition & 1 deletion charts/wandb-service/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ spec:
{{- end }}
securityContext:
{{- toYaml .Values.securityContext | nindent 12 }}
image: "{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}"
image: "{{ .Values.image.repository }}:{{ .Values.image.tag }}"
imagePullPolicy: {{ .Values.image.pullPolicy }}
{{ with .Values.service.ports }}
ports:
Expand Down
2 changes: 1 addition & 1 deletion charts/wandb-service/templates/statefulset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ spec:
- name: {{ .Chart.Name }}
securityContext:
{{- toYaml .Values.securityContext | nindent 12 }}
image: "{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}"
image: "{{ .Values.image.repository }}:{{ .Values.image.tag }}"
imagePullPolicy: {{ .Values.image.pullPolicy }}
ports:
- name: http
Expand Down

0 comments on commit 40b4d37

Please sign in to comment.