Add master NodeSelector to SNMP pods

SNMP pods should not be installed on worker nodes, since
trap generation will not work as expected.

This change adds a node-role.kubernetes.io/master NodeSelector
in order to force the pods to be Scheduled only on master nodes.

Closes-Bug: 1955571

Test Plan:

PASS: Verify that tolerations and NodeSelector are installed properly.

PASS: Add a NoSchedule taint to master nodes and verify that
pods are not being scheduled on worker nodes.

Signed-off-by: Enzo Candotti <enzo.candotti@windriver.com>
Change-Id: I28c279187463cfdc79fd39f40a817be4ee4cf011
This commit is contained in:
Enzo Candotti 2021-12-22 11:14:21 -03:00
parent 6bf825fdcb
commit 9913c1ad6c
2 changed files with 14 additions and 3 deletions

View File

@ -23,10 +23,14 @@ spec:
spec:
imagePullSecrets:
- name: default-registry-key
{{- with .Values.tolerations }}
tolerations:
- key: "node-role.kubernetes.io/master"
operator: "Exists"
effect: "NoSchedule"
{{ toYaml . | indent 6 }}
{{- end }}
{{- if .Values.nodeSelector }}
nodeSelector:
{{ .Values.nodeSelector | toYaml | trim | indent 8 }}
{{- end }}
containers:
- name: {{ .Chart.Name }}
image: "{{ .Values.snmp.image.repository }}:{{ .Values.snmp.image.tag }}"

View File

@ -30,6 +30,13 @@ image:
nameOverride: ""
fullnameOverride: ""
nodeSelector: { node-role.kubernetes.io/master: "" }
tolerations:
- key: "node-role.kubernetes.io/master"
operator: "Exists"
effect: "NoSchedule"
configmap:
name_snmpd: snmpd-etc-config
name_fm: snmpd-fm