Skip to content

Commit

Permalink
Merge pull request #14537 from ycliuhw/ci-fix
Browse files Browse the repository at this point in the history
#14537

Remove relation before model teardown because of bug: lp:1952221;
  • Loading branch information
jujubot committed Aug 30, 2022
2 parents 990188b + dcedc90 commit a5bc9c4
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion tests/suites/deploy/deploy_bundles.sh
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@ run_deploy_cmr_bundle() {

juju deploy easyrsa
wait_for "easyrsa" ".applications | keys[0]"
wait_for "active" '.applications["easyrsa"] | ."application-status".current'

juju offer easyrsa:client
juju add-model other
Expand All @@ -47,10 +48,15 @@ run_deploy_cmr_bundle() {
sed "s/{{BOOTSTRAPPED_JUJU_CTRL_NAME}}/${BOOTSTRAPPED_JUJU_CTRL_NAME}/g" "${bundle}" >"${TEST_DIR}/cmr_bundles_test_deploy.yaml"
juju deploy "${TEST_DIR}/cmr_bundles_test_deploy.yaml"

wait_for "active" '.applications["etcd"] | ."application-status".current'
wait_for "etcd" "$(idle_condition "etcd")"
wait_for "active" "$(workload_status "etcd" 0).current"

# TODO: no need to remove-relation before destroying model once we fixed(lp:1952221).
juju remove-relation etcd easyrsa

destroy_model "test-cmr-bundles-deploy"
destroy_model "other"
destroy_model "test-cmr-bundles-deploy"
}

# run_deploy_exported_charmstore_bundle_with_fixed_revisions tests how juju deploys
Expand Down

0 comments on commit a5bc9c4

Please sign in to comment.