openstack/magnum-capi-helm-charts
Code Issues Proposed changes

Ensure machine deployment annotations are always produced

Browse Source
This commit is contained in:
Matt Pryor 2023-02-01 10:59:44 +00:00
parent d88e1750d4
commit 5ff5db625c
1 changed files with 16 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
charts/openstack-cluster/templates/node-group/machine-deployment.yaml
View File

@ -8,27 +8,27 @@ metadata:
labels:
{{- include "openstack-cluster.componentLabels" (list $ "worker") | nindent 4 }}
{{ $.Values.projectPrefix }}/node-group: {{ $nodeGroup.name }}
{{- if $nodeGroup.autoscale }}
{{-
$machineCountMin := $nodeGroup.machineCountMin |
required (printf "no minimum machine count specified for node group '%s'" $nodeGroup.name)
}}
{{- if lt $machineCountMin 1.0 }}
{{- fail "autoscaling node groups with a minimum machine count < 1 are not currently supported" }}
{{- end }}
{{-
$machineCountMax := $nodeGroup.machineCountMax |
required (printf "no maximum machine count specified for node group '%s'" $nodeGroup.name)
}}
{{- if lt $machineCountMax $machineCountMin }}
{{- fail (printf "maximum machine count < minimum machine count for node group '%s'" $nodeGroup.name) }}
{{- end }}
annotations:
# Argo should manage this resource even though it has an owner ref for the cluster
argocd.argoproj.io/managed-resource: "true"
{{- if $nodeGroup.autoscale }}
{{-
$machineCountMin := $nodeGroup.machineCountMin |
required (printf "no minimum machine count specified for node group '%s'" $nodeGroup.name)
}}
{{- if lt $machineCountMin 1.0 }}
{{- fail "autoscaling node groups with a minimum machine count < 1 are not currently supported" }}
{{- end }}
{{-
$machineCountMax := $nodeGroup.machineCountMax |
required (printf "no maximum machine count specified for node group '%s'" $nodeGroup.name)
}}
{{- if lt $machineCountMax $machineCountMin }}
{{- fail (printf "maximum machine count < minimum machine count for node group '%s'" $nodeGroup.name) }}
{{- end }}
cluster.x-k8s.io/cluster-api-autoscaler-node-group-min-size: {{ quote $machineCountMin }}
cluster.x-k8s.io/cluster-api-autoscaler-node-group-max-size: {{ quote $machineCountMax }}
{{- end }}
{{- end }}
spec:
clusterName: {{ include "openstack-cluster.clusterName" $ }}
{{- if not $nodeGroup.autoscale }}