Skip to content

Commit

Permalink
chore: update health checks and mount the secret even if its not an e…
Browse files Browse the repository at this point in the history
…xternal secret
  • Loading branch information
MarshallAsch committed Jul 2, 2024
1 parent 0c6d9ec commit 51b7a03
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 5 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,8 @@ jobs:
type=raw,value=latest,enable={{is_default_branch}}
type=ref,event=branch,suffix=-{{sha}}
type=ref,event=pr
type=sha,prefix=
type=sha
type=sha,prefix=,enable={{is_default_branch}}
type=sha,enable={{is_default_branch}}
- name: Set up QEMU
uses: docker/setup-qemu-action@v3
Expand Down
2 changes: 1 addition & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
FROM node:20.13.0-alpine

ARG PORT=3000
ENV PORT=3000
WORKDIR /app
COPY . /app
RUN yarn install
Expand Down
2 changes: 0 additions & 2 deletions charts/music-notifications/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,11 +27,9 @@ spec:
- name: {{ .Chart.Name }}
image: "{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}"
imagePullPolicy: {{ .Values.image.pullPolicy }}
{{- if .Values.secret.externalSecretKey }}
envFrom:
- secretRef:
name: {{ include "music-notifications.fullname" . }}
{{- end }}
livenessProbe:
failureThreshold: 1
httpGet:
Expand Down

0 comments on commit 51b7a03

Please sign in to comment.