From 9af73abe0134ca02a5e37f423857d91c60c5b0b3 Mon Sep 17 00:00:00 2001 From: Paul Czarkowski Date: Thu, 14 Jul 2022 11:13:29 -0500 Subject: [PATCH] fix gpu chart to actually work Signed-off-by: Paul Czarkowski --- charts/aro-gpu/Chart.yaml | 2 +- charts/aro-gpu/README.md | 4 ++-- charts/aro-gpu/templates/NOTES.txt | 8 +++++++- 3 files changed, 10 insertions(+), 4 deletions(-) diff --git a/charts/aro-gpu/Chart.yaml b/charts/aro-gpu/Chart.yaml index 68bde98..1a98d90 100644 --- a/charts/aro-gpu/Chart.yaml +++ b/charts/aro-gpu/Chart.yaml @@ -2,7 +2,7 @@ apiVersion: v2 name: aro-gpu description: A Helm chart for Deploying GPU-enabled machinesets in Azure. type: application -version: 0.1.0 +version: 0.1.1 maintainers: - name: paulczar home: https://github.com/rh-mobb/helm-charts diff --git a/charts/aro-gpu/README.md b/charts/aro-gpu/README.md index 6b40f44..b753414 100644 --- a/charts/aro-gpu/README.md +++ b/charts/aro-gpu/README.md @@ -29,8 +29,8 @@ This Helm chart will deploy GPU nodes on ARO clusters. 1. Install a Chart ```bash - helm upgrade --install -n $NAMESPACE openshift-cluster-api \ - mobb/aro-gpu + helm upgrade --install -n $NAMESPACE openshift-machine-api \ + gpu mobb/aro-gpu ``` 1. Wait for the new GPU nodes to be available diff --git a/charts/aro-gpu/templates/NOTES.txt b/charts/aro-gpu/templates/NOTES.txt index dd91dcf..fab64e9 100644 --- a/charts/aro-gpu/templates/NOTES.txt +++ b/charts/aro-gpu/templates/NOTES.txt @@ -1,4 +1,10 @@ {{- $cluster := lookup "aro.openshift.io/v1alpha1" "Cluster" "" "cluster" -}} -{{- if not (empty $cluster.spec) -}} +{{- if (empty $cluster.spec) -}} Could not proceed as chart does not support tempate or --dry-run. +{{- else }} +Check that the new MachineSet is creating: + $ oc get machineset {{ $cluster.spec.infraId }}-{{ include "aro-gpu.fullname" . }}-{{ $cluster.spec.location }} + +Check on the Created machines: + $ oc get machines oc get machines --selector "machine.openshift.io/cluster-api-machineset={{ $cluster.spec.infraId }}-{{ include "aro-gpu.fullname" . }}-{{ $cluster.spec.location }} {{- end -}}