Skip to content

Commit

Permalink
fix: frontend PodDisruptionBudget (#633)
Browse files Browse the repository at this point in the history
### 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.
  • Loading branch information
filiphaftek authored Feb 11, 2025
1 parent 8a0dc9f commit 4ba75fb
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 2 deletions.
2 changes: 2 additions & 0 deletions charts/sourcegraph/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@ Use `**BREAKING**:` to denote a breaking change

## Unreleased

- Fix Pod Disruption Budget for sourcegraph-frontend

## 5.10.0

- Updated OpenTelemetry collector and agent images to run as non-root users [#543](https://github.com/sourcegraph/deploy-sourcegraph-helm/pull/543)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ metadata:
spec:
{{- toYaml .Values.frontend.podDisruptionBudget | nindent 2 }}
selector:
{{- include "sourcegraph.selectorLabels" . | nindent 4 }}
app: sourcegraph-frontend
matchLabels:
{{- include "sourcegraph.selectorLabels" . | nindent 6 }}
app: sourcegraph-frontend
{{- end }}
3 changes: 3 additions & 0 deletions charts/sourcegraph/tests/podDisruptionBudget_test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,3 +13,6 @@ tests:
- equal:
path: spec.minAvailable
value: 1
- equal:
path: spec.selector.matchLabels.app
value: sourcegraph-frontend

0 comments on commit 4ba75fb

Please sign in to comment.