Unverified Commit a35df0f5 authored by Ran Rubin's avatar Ran Rubin Committed by GitHub
Browse files

chore: fix deploy test (#6623)

parent db88c953
...@@ -341,7 +341,9 @@ jobs: ...@@ -341,7 +341,9 @@ jobs:
cd deploy/helm/charts/platform/ cd deploy/helm/charts/platform/
helm dep build . helm dep build .
# Install platform with namespace restriction for single profile testing # Install platform with namespace restriction for single profile testing
# we manage crds via Velonix so we skip the crds installation
helm upgrade --install dynamo-platform . --namespace ${NAMESPACE} \ helm upgrade --install dynamo-platform . --namespace ${NAMESPACE} \
--skip-crds \
--set dynamo-operator.namespaceRestriction.enabled=true \ --set dynamo-operator.namespaceRestriction.enabled=true \
--set dynamo-operator.namespaceRestriction.allowedNamespaces[0]=${NAMESPACE} \ --set dynamo-operator.namespaceRestriction.allowedNamespaces[0]=${NAMESPACE} \
--set dynamo-operator.controllerManager.manager.image.repository=${{ secrets.AZURE_ACR_HOSTNAME }}/ai-dynamo/dynamo \ --set dynamo-operator.controllerManager.manager.image.repository=${{ secrets.AZURE_ACR_HOSTNAME }}/ai-dynamo/dynamo \
...@@ -509,10 +511,8 @@ jobs: ...@@ -509,10 +511,8 @@ jobs:
cleanup: cleanup:
name: Cleanup AKS resources name: Cleanup AKS resources
runs-on: prod-default-small-v2 runs-on: prod-default-small-v2
if: >- if: always()
always() && needs: [deploy-operator, deploy-test-trtllm, deploy-test-sglang, deploy-test-vllm]
needs.deploy-status-check.result == 'success'
needs: [deploy-operator, deploy-status-check]
steps: steps:
- name: Checkout code - name: Checkout code
uses: actions/checkout@08eba0b27e820071cde6df949e0beb9ba4906955 # v4.3.0 uses: actions/checkout@08eba0b27e820071cde6df949e0beb9ba4906955 # v4.3.0
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment