Skip to content

Commit 5eaa6e0

Browse files
committed
fix: dont fail silently on awscli errors during artifact setup
If something goes wrong during aws s3 cp, exit early with an error message instead of quietly trying to soldier on. A common(-ish) cause of something going wrong here is having installed the aws cli for the wrong CPU architecture (e.g. x86 AWSCLi on ARM64), as the awscli installer apparently doesnt not check the architecture, and its easy to copy the wrong command from the website. Signed-off-by: Patrick Roy <[email protected]>
1 parent 51a324a commit 5eaa6e0

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

tools/devtool

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -575,6 +575,7 @@ ensure_ci_artifacts() {
575575
if [ ! -d "$ARTIFACTS" ]; then
576576
mkdir -pv $ARTIFACTS
577577
aws s3 sync --no-sign-request "$S3_URL" "$ARTIFACTS"
578+
ok_or_die "Failed to download CI artifacts using awscli!"
578579
cmd_sh "./tools/setup-ci-artifacts.sh"
579580
fi
580581
}

0 commit comments

Comments
 (0)