airship
/
promenade
Code Issues Proposed changes

Add "labels" to apiserver-webhook deployment 

Defines a "labels" variable and adds it to the deployment itself. In
addition, reuses said labels variable to replace other parts of the
deployment in which this was repeated. This will clean up the chart
a bit and enable Armada itself to properly wait for certain percentages
of the deployment replicas to be ready prior to proceeding. Prior to
this change, there wasn't a way to select the apiserver-webhook
deployment via labels.

Change-Id: If1ddb4feb8cde414e76a412431222f8f608f3b18
This commit is contained in:
DeJaeger, Darren (dd118r) 2021-04-30 16:55:11 -04:00
parent 300a399aff
commit 774f85248d
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
charts/apiserver-webhook/templates/deployment.yaml
View File

@ -99,24 +99,25 @@ tls-{{ $group | replace "_" "-" }}-{{ $type | replace "_" "-" }}
{{- $mounts_apiserver := .Values.pod.mounts.apiserver_webhook.apiserver }}
{{- $mounts_webhook := .Values.pod.mounts.apiserver_webhook.webhook }}
{{- $envAll := . }}
{{- $labels := tuple $envAll "kubernetes-keystone-webhook" "api" | include "helm-toolkit.snippets.kubernetes_metadata_labels" -}}
---
apiVersion: apps/v1
kind: Deployment
metadata:
name: {{ .Release.Name }}-apiserver-webhook
labels:
{{ $labels | indent 4 }}
annotations:
{{ tuple $envAll | include "helm-toolkit.snippets.release_uuid" | indent 4 }}
{{ tuple $envAll "kubernetes-keystone-webhook" "api" | include "helm-toolkit.snippets.kubernetes_metadata_labels" | indent 4 }}
spec:
replicas: {{ $envAll.Values.pod.replicas.api }}
selector:
matchLabels:
{{ tuple $envAll "kubernetes-keystone-webhook" "api" | include "helm-toolkit.snippets.kubernetes_metadata_labels" | indent 6 }}
{{ $labels | indent 6 }}
template:
metadata:
labels:
{{ tuple $envAll "kubernetes-keystone-webhook" "api" | include "helm-toolkit.snippets.kubernetes_metadata_labels" | indent 8 }}
{{ $labels | indent 8 }}
annotations:
{{ tuple $envAll | include "helm-toolkit.snippets.release_uuid" | indent 8 }}
{{ dict "envAll" $envAll "podName" "apiserver-webhook" "containerNames" (list "apiserver" "webhook") | include "helm-toolkit.snippets.kubernetes_mandatory_access_control_annotation" | indent 8 }}