fix(t8s-cluster/management-cluster): wrong check for gpu instances

teutonet · Aug 23, 2023 · 269afac · 269afac
1 parent 10d7b8c
commit 269afac
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/...management-cluster/clusterClass/kubeadmConfigTemplate/gpuWorkerKubeadmConfigTemplate.yaml b/...management-cluster/clusterClass/kubeadmConfigTemplate/gpuWorkerKubeadmConfigTemplate.yaml
@@ -1,4 +1,4 @@
-{{- if (include "t8s-cluster.hasGPUNodes" .) }}
+{{- if eq (include "t8s-cluster.hasGPUNodes" .) "true" }}
 apiVersion: bootstrap.cluster.x-k8s.io/v1beta1
 kind: KubeadmConfigTemplate
 metadata:

diff --git a/charts/t8s-cluster/templates/workload-cluster/gpu-operator.yaml b/charts/t8s-cluster/templates/workload-cluster/gpu-operator.yaml
@@ -1,4 +1,4 @@
-{{- if (include "t8s-cluster.hasGPUNodes" .) }}
+{{- if eq (include "t8s-cluster.hasGPUNodes" .) "true" }}
 apiVersion: helm.toolkit.fluxcd.io/v2beta1
 kind: HelmRelease
 metadata:

diff --git a/charts/t8s-cluster/values.yaml b/charts/t8s-cluster/values.yaml
@@ -10,7 +10,7 @@ global:
       url: https://helm.ngc.nvidia.com/nvidia
       charts:
         gpu-operator: 23.x.x
-      condition: '{{ include "t8s-cluster.hasGPUNodes" . }}'
+      condition: '{{ eq (include "t8s-cluster.hasGPUNodes" .) "true" }}'
     cloud-provider-openstack:
       url: https://kubernetes.github.io/cloud-provider-openstack
     cetic: