starlingx/audit-armada-app
Code Issues Proposed changes

switch to use new K8s control-plane label

Browse Source 
The node-role.kubernetes.io/master= label is being deprecated and is
being replaced by node-role.kubernetes.io/control-plane=

Similarly the node-role.kubernetes.io/master:NoSchedule taint is being
replaced by node-role.kubernetes.io/control-plane:NoSchedule

We need to update the code appropriately.  We can switch to the new
label since it's already available, but where we have a toleration for
the existing taint we need to keep the existing toleration and add
another one for what will be the new taint.

Test Plan:
PASS: Applied application in standard lab with additional
      "control-plane" taint, ensured application pods ran on
      controller nodes.

Story: 2010301
Task: 46575

Signed-off-by: Chris Friesen <chris.friesen@windriver.com>
Change-Id: Iadc333f6a3fb9bf69c6d72e7cc7c6e74266e17e9
This commit is contained in:
Chris Friesen 2022-10-14 17:54:04 -04:00
parent 475e9b98fe
commit b0ebdf6cfa
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
stx-audit-helm/stx-audit-helm/helm-charts/audit/templates/daemonset.yaml
View File

@ -27,6 +27,9 @@ spec:
- key: node-role.kubernetes.io/master
operator: Exists
effect: NoSchedule
- key: node-role.kubernetes.io/control-plane
operator: Exists
effect: NoSchedule
containers:
- name: {{ include "audit.name" . }}
image: "{{ .Values.image.repository }}:{{ .Values.image.tag }}"