Skip to content

Commit

Permalink
fix(dependencies): install charmcraft if missing in test-runner script
Browse files Browse the repository at this point in the history
Ensure charmcraft is installed before running integration tests, similar to how jq is handled.
This should fix several red test in JUJU gating ci :

* tests/suites/deploy
* tests/suites/hooks
* tests/suites/relations
* tests/suites/sidecar
* tests/suites/spaces_ec2
* tests/suites/storage
  • Loading branch information
gfouillet committed Dec 17, 2024
1 parent cc154f6 commit 6bedf18
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions jobs/ci-run/integration/common/test-runner.sh
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,9 @@ sudo apt-get -y update
# set, so retries succeed.
attempts=0
while [ $attempts -lt 3 ]; do
if ! which charmcraft >/dev/null 2>&1; then
sudo snap install charmcraft --classic || true
fi
if ! which jq >/dev/null 2>&1; then
sudo snap install jq || true
fi
Expand Down

0 comments on commit 6bedf18

Please sign in to comment.