Skip to content

Commit

Permalink
ci: add SKU type for the created cluster (#152)
Browse files Browse the repository at this point in the history
Signed-off-by: Heba <[email protected]>
  • Loading branch information
helayoty authored Aug 6, 2024
1 parent d802f6e commit d455817
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 5 deletions.
4 changes: 2 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -62,9 +62,9 @@ az-mkacr: az-mkrg ## Create test ACR
az acr login --name $(AZURE_ACR_NAME)

az-mkaks: az-mkacr ## Create test AKS cluster (with msi, oidc and workload identity enabled)
az aks create --name $(AZURE_CLUSTER_NAME) --resource-group $(AZURE_RESOURCE_GROUP) --attach-acr $(AZURE_ACR_NAME) \
az aks create --name $(AZURE_CLUSTER_NAME) --resource-group $(AZURE_RESOURCE_GROUP) --attach-acr $(AZURE_ACR_NAME) \
--kubernetes-version $(AKS_K8S_VERSION) --node-count 1 --generate-ssh-keys \
--enable-managed-identity --enable-workload-identity --enable-oidc-issuer -o none
--enable-managed-identity --enable-workload-identity --enable-oidc-issuer --node-vm-size Standard_D2s_v3 -o none
az aks get-credentials --name $(AZURE_CLUSTER_NAME) --resource-group $(AZURE_RESOURCE_GROUP)

az-rmrg: ## Destroy test ACR and AKS cluster by deleting the resource group (use with care!)
Expand Down
3 changes: 0 additions & 3 deletions hack/deploy/configure-helm-values.sh
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,6 @@ AZURE_RESOURCE_GROUP_MC=$(jq -r ".nodeResourceGroup" <<< "$AKS_JSON")
AZURE_TENANT_ID=$(az account show -o json |jq -r ".tenantId")
AZURE_SUBSCRIPTION_ID=$(az account show -o json |jq -r ".id")




GPU_PROVISIONER_USER_ASSIGNED_CLIENT_ID=$(az identity show --resource-group "${AZURE_RESOURCE_GROUP}" --name "${AZURE_GPU_PROVISIONER_USER_ASSIGNED_IDENTITY_NAME}" --query 'clientId' -otsv)

export CLUSTER_NAME AZURE_LOCATION AZURE_RESOURCE_GROUP_MC GPU_PROVISIONER_USER_ASSIGNED_CLIENT_ID AZURE_TENANT_ID AZURE_SUBSCRIPTION_ID
Expand Down

0 comments on commit d455817

Please sign in to comment.