Skip to content

Commit da70e0d

Browse files
filiphaftekenriquegh
authored andcommitted
fix: frontend PodDisruptionBudget (#633)
### Checklist - [x] Follow the [manual testing process](https://github.com/sourcegraph/deploy-sourcegraph-helm/blob/main/TEST.md) - [x] Update [changelog](https://github.com/sourcegraph/deploy-sourcegraph-helm/blob/main/charts/sourcegraph/CHANGELOG.md) - [x] Update [Kubernetes update doc](https://docs.sourcegraph.com/admin/updates/kubernetes) ### Test plan Unit test. e2e on cluster.
1 parent b27dd47 commit da70e0d

File tree

3 files changed

+8
-2
lines changed

3 files changed

+8
-2
lines changed

charts/sourcegraph/CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,8 @@ Use `**BREAKING**:` to denote a breaking change
88

99
## Unreleased
1010

11+
- Fix Pod Disruption Budget for sourcegraph-frontend
12+
1113
## 5.10.0
1214

1315
- Updated OpenTelemetry collector and agent images to run as non-root users [#543](https://github.com/sourcegraph/deploy-sourcegraph-helm/pull/543)

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

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ metadata:
66
spec:
77
{{- toYaml .Values.frontend.podDisruptionBudget | nindent 2 }}
88
selector:
9-
{{- include "sourcegraph.selectorLabels" . | nindent 4 }}
10-
app: sourcegraph-frontend
9+
matchLabels:
10+
{{- include "sourcegraph.selectorLabels" . | nindent 6 }}
11+
app: sourcegraph-frontend
1112
{{- end }}

charts/sourcegraph/tests/podDisruptionBudget_test.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,3 +13,6 @@ tests:
1313
- equal:
1414
path: spec.minAvailable
1515
value: 1
16+
- equal:
17+
path: spec.selector.matchLabels.app
18+
value: sourcegraph-frontend

0 commit comments

Comments
 (0)