Skip to content

Commit 9abe9e1

Browse files
Cache Proxy: add app: <fullname> label to deployment pods and selector
Adds an `app` label matching the chart fullname to both the cache-proxy pod template and the Deployment's `matchLabels` selector. Note that `matchLabels` is immutable on existing Deployments, so an upgrade from a release without this label requires deleting and recreating the Deployment. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 9a6be4e commit 9abe9e1

2 files changed

Lines changed: 3 additions & 1 deletion

File tree

charts/buildbuddy-enterprise-cache-proxy/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
apiVersion: v1
22
description: BuildBuddy Enterprise Cache Proxy
33
name: buildbuddy-enterprise-cache-proxy
4-
version: 0.0.3 # Chart version
4+
version: 0.0.4 # Chart version
55
appVersion: 2.261.0 # Version of deployed cache proxy
66
keywords:
77
- buildbuddy

charts/buildbuddy-enterprise-cache-proxy/templates/deployment.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ spec:
1616
{{- .Values.strategy | toYaml | nindent 4 }}
1717
selector:
1818
matchLabels:
19+
app: {{ include "buildbuddy.fullname" . }}
1920
app.kubernetes.io/name: {{ include "buildbuddy.name" . }}
2021
app.kubernetes.io/instance: {{ .Release.Name }}
2122
{{- if .Values.minReadySeconds }}
@@ -31,6 +32,7 @@ spec:
3132
{{- .Values.extraPodAnnotations | toYaml | nindent 8 }}
3233
{{- end }}
3334
labels:
35+
app: {{ include "buildbuddy.fullname" . }}
3436
app.kubernetes.io/name: {{ include "buildbuddy.name" . }}
3537
app.kubernetes.io/instance: {{ .Release.Name }}
3638
app.kubernetes.io/managed-by: {{ .Release.Service }}

0 commit comments

Comments
 (0)