Merge pull request #19 from fiverr/consolidating_fix

removing consolidated conditions
fiverr · Dec 3, 2024 · ed112f8 · ed112f8
2 parents 6b356d6 + f2fd409
commit ed112f8
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/charts/karpenter_nodes/Chart.yaml b/charts/karpenter_nodes/Chart.yaml
@@ -1,6 +1,6 @@
 apiVersion: v2
 name: karpenter_nodes
-version: 1.1.4
+version: 1.1.5
 description: A Helm chart for generating NodeClasses and NodePools for Karpenter
 maintainers:
   - name: nadavbuc
diff --git a/charts/karpenter_nodes/templates/nodepool.yaml b/charts/karpenter_nodes/templates/nodepool.yaml
@@ -156,9 +156,7 @@ spec:
       expireAfter: {{ $v.expireAfter | default $.Values.expireAfter }}
   disruption:
     consolidationPolicy: {{$v.consolidationPolicy | default $.Values.consolidationPolicy}}
-    {{- if eq ($v.consolidationPolicy | default $.Values.consolidationPolicy) "WhenEmpty" }}
     consolidateAfter: {{ $v.consolidateAfter | default $.Values.consolidateAfter }}
-    {{- end }}
     {{- if $v.budgets }}
     budgets:
     {{- toYaml $v.budgets | nindent 6 }}