Skip to content

Commit 294ac78

Browse files
authored
Merge pull request #23 from stackhpc/fix/api-deployment-env
Fix token templating in API deployment
2 parents b5d189e + 00d4865 commit 294ac78

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

chart/templates/api/deployment.yml

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -35,15 +35,14 @@ spec:
3535
{{- if .Values.huggingface.secretName }}
3636
envFrom:
3737
- secretRef:
38-
name: huggingface-token
39-
{{- else if .Values.huggingface.token }}
38+
name: {{ .Values.huggingface.secretName }}
39+
{{- end }}
4040
env:
4141
- name: DO_NOT_TRACK
42-
value: 1
42+
value: "1"
43+
{{- if .Values.huggingface.token }}
4344
- name: HUGGING_FACE_HUB_TOKEN
4445
value: {{ quote .Values.huggingface.token }}
45-
{{- else if contains "Llama" .Values.huggingface.model -}}
46-
{{- fail "Either secretName or token value must be set for Llama and other gated models" }}
4746
{{- end }}
4847
readinessProbe:
4948
httpGet:
@@ -60,4 +59,4 @@ spec:
6059
- name: shm
6160
emptyDir:
6261
medium: Memory
63-
sizeLimit: 1Gi
62+
sizeLimit: 1Gi

0 commit comments

Comments
 (0)