Skip to content

Commit

Permalink
fix: add node tolerations to deployments (#321)
Browse files Browse the repository at this point in the history
While the observability-operator, obo-prometheus-operator and
obo-prometheus-operator-admission-webhook deployments are configured to
be scheduled preferably on infra nodes, they lacked the required
toleration that would enable them being scheduled properly.

This change fixes the issue by adding the expected toleration.

Signed-off-by: Simon Pasquier <spasquie@redhat.com>
  • Loading branch information
simonpasquier authored Aug 7, 2023
1 parent 6c09f46 commit d0ba3a9
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 0 deletions.
12 changes: 12 additions & 0 deletions bundle/manifests/observability-operator.clusterserviceversion.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -448,6 +448,10 @@ spec:
runAsNonRoot: true
runAsUser: 65534
serviceAccountName: obo-prometheus-operator
tolerations:
- effect: NoSchedule
key: node-role.kubernetes.io/infra
operator: Exists
- label:
app.kubernetes.io/name: prometheus-operator-admission-webhook
app.kubernetes.io/part-of: observability-operator
Expand Down Expand Up @@ -517,6 +521,10 @@ spec:
runAsNonRoot: true
runAsUser: 65534
serviceAccountName: obo-prometheus-operator-admission-webhook
tolerations:
- effect: NoSchedule
key: node-role.kubernetes.io/infra
operator: Exists
- label:
app.kubernetes.io/component: operator
app.kubernetes.io/name: observability-operator
Expand Down Expand Up @@ -579,6 +587,10 @@ spec:
runAsNonRoot: true
serviceAccountName: observability-operator-sa
terminationGracePeriodSeconds: 30
tolerations:
- effect: NoSchedule
key: node-role.kubernetes.io/infra
operator: Exists
strategy: deployment
installModes:
- supported: false
Expand Down
4 changes: 4 additions & 0 deletions deploy/olm/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,10 @@ patches:
- key: node-role.kubernetes.io/infra
operator: Exists
weight: 1
tolerations:
- effect: NoSchedule
key: node-role.kubernetes.io/infra
operator: Exists
target:
group: apps
kind: Deployment
Expand Down

0 comments on commit d0ba3a9

Please sign in to comment.