Skip to content

Commit 6de65bd

Browse files
committed
Fix prereqs-bucket.sh: handle bucket-exists-but-no-stack, fix template path
1 parent 2f5fc9d commit 6de65bd

1 file changed

Lines changed: 17 additions & 13 deletions

File tree

tuts/000-prereqs-bucket/prereqs-bucket.sh

Lines changed: 17 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -21,24 +21,28 @@ if [ "$STATUS" = "CREATE_COMPLETE" ] || [ "$STATUS" = "UPDATE_COMPLETE" ]; then
2121
fi
2222

2323
echo "Creating bucket: $BUCKET_NAME"
24-
if [ "$REGION" = "us-east-1" ]; then
25-
aws s3api create-bucket --bucket "$BUCKET_NAME"
24+
if aws s3api head-bucket --bucket "$BUCKET_NAME" 2>/dev/null; then
25+
echo "Bucket already exists: $BUCKET_NAME"
2626
else
27-
aws s3api create-bucket --bucket "$BUCKET_NAME" \
28-
--create-bucket-configuration LocationConstraint="$REGION"
27+
if [ "$REGION" = "us-east-1" ]; then
28+
aws s3api create-bucket --bucket "$BUCKET_NAME"
29+
else
30+
aws s3api create-bucket --bucket "$BUCKET_NAME" \
31+
--create-bucket-configuration LocationConstraint="$REGION"
32+
fi
33+
34+
aws s3api put-bucket-encryption --bucket "$BUCKET_NAME" \
35+
--server-side-encryption-configuration \
36+
'{"Rules":[{"ApplyServerSideEncryptionByDefault":{"SSEAlgorithm":"AES256"}}]}'
37+
38+
aws s3api put-public-access-block --bucket "$BUCKET_NAME" \
39+
--public-access-block-configuration \
40+
'BlockPublicAcls=true,BlockPublicPolicy=true,IgnorePublicAcls=true,RestrictPublicBuckets=true'
2941
fi
3042

31-
aws s3api put-bucket-encryption --bucket "$BUCKET_NAME" \
32-
--server-side-encryption-configuration \
33-
'{"Rules":[{"ApplyServerSideEncryptionByDefault":{"SSEAlgorithm":"AES256"}}]}'
34-
35-
aws s3api put-public-access-block --bucket "$BUCKET_NAME" \
36-
--public-access-block-configuration \
37-
'BlockPublicAcls=true,BlockPublicPolicy=true,IgnorePublicAcls=true,RestrictPublicBuckets=true'
38-
3943
echo "Registering bucket with CloudFormation stack: $STACK_NAME"
4044
aws cloudformation deploy \
41-
--template-file "$SCRIPT_DIR/prereq-bucket.yaml" \
45+
--template-file "$SCRIPT_DIR/cfn-prereqs-bucket.yaml" \
4246
--stack-name "$STACK_NAME" \
4347
--parameter-overrides "BucketName=$BUCKET_NAME"
4448

0 commit comments

Comments
 (0)