From cc8e02d0fa109fe7c59c00856dda880864d9065e Mon Sep 17 00:00:00 2001
From: Heba <31887807+helayoty@users.noreply.github.com>
Date: Wed, 8 May 2024 09:13:51 -0700
Subject: [PATCH] docs: Update gpu-provisioner installation step (#374)

**Reason for Change**:
Update the installation doc with the new `gpu-provisoner` helm chart
install.

**Requirements**

- [ ] added unit tests and e2e tests (if applicable).

**Issue Fixed**:
<!-- If this PR fixes GitHub issue 4321, add "Fixes #4321" to the next
line. -->

**Notes for Reviewers**:

---------

Signed-off-by: Heba <31887807+helayoty@users.noreply.github.com>
---
 docs/installation.md | 38 +++++++-------------------------------
 1 file changed, 7 insertions(+), 31 deletions(-)

diff --git a/docs/installation.md b/docs/installation.md
index e441d1cfe..111042c0c 100644
--- a/docs/installation.md
+++ b/docs/installation.md
@@ -76,37 +76,13 @@ az role assignment create --assignee $IDENTITY_PRINCIPAL_ID --scope /subscriptio
 Install the Node provisioner controller.
 ```bash
 # get additional values for helm chart install
-export NODE_RESOURCE_GROUP=$(az aks show -n $MY_CLUSTER -g $RESOURCE_GROUP --query nodeResourceGroup -o tsv)
-export LOCATION=$(az aks show -n $MY_CLUSTER -g $RESOURCE_GROUP --query location -o tsv)
-export TENANT_ID=$(az account show --query tenantId -o tsv)
-
-# create a local values override file
-cat << EOF > values.override.yaml
-controller:
-  env:
-  - name: ARM_SUBSCRIPTION_ID
-    value: $SUBSCRIPTION
-  - name: LOCATION
-    value: $LOCATION
-  - name: AZURE_CLUSTER_NAME
-    value: $MY_CLUSTER
-  - name: AZURE_NODE_RESOURCE_GROUP
-    value: $NODE_RESOURCE_GROUP
-  - name: ARM_RESOURCE_GROUP
-    value: $RESOURCE_GROUP
-  - name: LEADER_ELECT
-    value: "false"
-workloadIdentity:
-  clientId: $IDENTITY_CLIENT_ID
-  tenantId: $TENANT_ID
-settings:
-  azure:
-    clusterName: $MY_CLUSTER
-EOF
-
-# install gpu-provisioner using values override file
-helm install gpu-provisioner ./charts/kaito/gpu-provisioner \
---namespace gpu-provisioner --create-namespace -f values.override.yaml
+export GPU_PROVISIONER_VERSION=0.2.0
+
+curl -sO https://raw.githubusercontent.com/Azure/gpu-provisioner/main/hack/deploy/configure-helm-values.sh
+chmod +x ./configure-helm-values.sh && ./configure-helm-values.sh $MY_CLUSTER $RESOURCE_GROUP $IDENTITY_NAME
+
+helm install gpu-provisioner --values gpu-provisioner-values.yaml --set settings.azure.clusterName=$MY_CLUSTER --wait \
+https://github.com/Azure/gpu-provisioner/raw/gh-pages/charts/gpu-provisioner-$GPU_PROVISIONER_VERSION.tgz
 ```
 
 #### Create the federated credential