Skip to content

Commit

Permalink
Merge pull request #15657 from anvial/JUJU-3842-fix-deploy-aks-clean-…
Browse files Browse the repository at this point in the history
…up-stage

#15657

This PR adds an explicit definition that deploy_aks used the `k8s` bootstrap provider. That will avoid errors on clean-up stage.

## Checklist

- [x] Code style: imports ordered, good names, simple structure, etc
- [x] Comments saying why design decisions were made
- [x] Go unit tests, with comments saying what you're testing

## QA steps


```sh
cd tests
./main.sh -v deploy_aks
```
  • Loading branch information
jujubot committed May 25, 2023
2 parents 0bb6df2 + 682a4b0 commit f16dd80
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions tests/suites/deploy_aks/task.sh
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,12 @@ test_deploy_aks() {
az aks create -g "${resource_group_name}" -n aks-cluster --generate-ssh-keys
juju add-k8s --aks --client --resource-group "${resource_group_name}" --storage test-aks-storage --cluster-name aks-cluster aks-k8s-cloud

export BOOTSTRAP_PROVIDER=k8s
bootstrap_custom_controller "test-deploy-aks" "aks-k8s-cloud"

test_deploy_aks_charms

export KILL_CONTROLLER=true
destroy_controller "test-deploy-aks"

juju remove-k8s --client aks-k8s-cloud
Expand Down

0 comments on commit f16dd80

Please sign in to comment.