Skip to content

Commit f36cff3

Browse files
authored
frontend: remove unused CACHE_DIR (#169)
The code backing this variable has been unused for atleast a year. Removing this also allows us to simplify deployment, since right now we tell customers to assign ephemeral storage to frontend (but that will never be used!) Test Plan: CI. I previously did a lot of validation that we can remove this (reading code, inspecting prod and dogfood environments for cache dir usage)
1 parent b7ab038 commit f36cff3

File tree

1 file changed

+0
-13
lines changed

1 file changed

+0
-13
lines changed

charts/sourcegraph/templates/frontend/sourcegraph-frontend.Deployment.yaml

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -84,15 +84,6 @@ spec:
8484
{{- include "sourcegraph.databaseAuth" (list . "codeIntelDB" "CODEINTEL_PG") | nindent 8 }}
8585
{{- include "sourcegraph.databaseAuth" (list . "codeInsightsDB" "CODEINSIGHTS_PG") | nindent 8 }}
8686
{{- include "sourcegraph.redisConnection" .| nindent 8 }}
87-
# POD_NAME is used by CACHE_DIR
88-
- name: POD_NAME
89-
valueFrom:
90-
fieldRef:
91-
fieldPath: metadata.name
92-
# CACHE_DIR stores larger items we cache. Majority of it is zip
93-
# archives of repositories at a commit.
94-
- name: CACHE_DIR
95-
value: /mnt/cache/$(POD_NAME)
9687
{{- if .Values.minio.enabled }}
9788
- name: PRECISE_CODE_INTEL_UPLOAD_AWS_ACCESS_KEY_ID
9889
valueFrom:
@@ -134,8 +125,6 @@ spec:
134125
securityContext:
135126
{{- toYaml .Values.frontend.containerSecurityContext | nindent 10 }}
136127
volumeMounts:
137-
- mountPath: /mnt/cache
138-
name: cache-ssd
139128
- mountPath: /home/sourcegraph
140129
name: home-dir
141130
{{- if .Values.frontend.extraVolumeMounts }}
@@ -156,8 +145,6 @@ spec:
156145
{{- end }}
157146
{{- include "sourcegraph.renderServiceAccountName" (list . "frontend") | trim | nindent 6 }}
158147
volumes:
159-
- emptyDir: {}
160-
name: cache-ssd
161148
- emptyDir: {}
162149
name: home-dir
163150
{{- if .Values.frontend.extraVolumes }}

0 commit comments

Comments
 (0)