Skip to content

Commit

Permalink
Merge pull request #23 from rh-mobb/fix-gpu-chart
Browse files Browse the repository at this point in the history
fix gpu chart to actually work
  • Loading branch information
paulczar authored Jul 14, 2022
2 parents f5e1372 + 9af73ab commit d97ec14
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 4 deletions.
2 changes: 1 addition & 1 deletion charts/aro-gpu/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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
4 changes: 2 additions & 2 deletions charts/aro-gpu/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
8 changes: 7 additions & 1 deletion charts/aro-gpu/templates/NOTES.txt
Original file line number Diff line number Diff line change
@@ -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 -}}

0 comments on commit d97ec14

Please sign in to comment.