Skip to content

Commit

Permalink
test: Print error message if aws cli not available
Browse files Browse the repository at this point in the history
Currently, devtool test silently fails if aws cli is not installed,
which results in CI artifacts not getting downloaded, which results in
all tests being skipped. Since the error message from downloading
artifacts is burried in other output, explicitly abort.

Signed-off-by: Patrick Roy <[email protected]>
  • Loading branch information
roypat committed Sep 13, 2023
1 parent 09ae793 commit bee2437
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions tools/devtool
Original file line number Diff line number Diff line change
Expand Up @@ -506,6 +506,10 @@ cmd_distclean() {
}

ensure_ci_artifacts() {
if [ ! command -v aws >/dev/null ]; then
die "AWS CLI not installed, which is required for downloading artifacts for integration tests."
fi

# Fetch all the artifacts so they are local
say "Fetching CI artifacts from S3"
S3_URL=s3://spec.ccfc.min/firecracker-ci/v1.5/$(uname -m)
Expand Down

0 comments on commit bee2437

Please sign in to comment.