Skip to content

Commit 0925d2c

Browse files
authored
Ce 2642 adding s3 bucket for mwaa etl scripts (#2064)
* creating s3 bucket for mwaa etl scripts * adding missing compliance exclude * adding missing aws_s3_bucket_public_access_block block * fixed typo * fixing typo
1 parent 4caf177 commit 0925d2c

File tree

2 files changed

+28
-0
lines changed

2 files changed

+28
-0
lines changed

terraform/compliance/s3.feature

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ Feature: S3
44
@exclude_module.qlik_server\[0\].aws_s3_bucket.qlik_alb_logs\[0\]
55
@exclude_module.airflow.aws_s3_bucket.bucket
66
@exclude_aws_s3_bucket.mwaa_bucket
7+
@exclude_aws_s3_bucket.mwaa_etl_scripts_bucket
78
Scenario: Data must be encrypted at rest for buckets created using server_side_encryption_configuration property within bucket resource
89
Given I have aws_s3_bucket defined
910
Then it must have server_side_encryption_configuration

terraform/core/46-mwaa-bucket-kms.tf

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,3 +89,30 @@ resource "aws_s3_bucket_public_access_block" "mwaa_bucket_block" {
8989
ignore_public_acls = true
9090
restrict_public_buckets = true
9191
}
92+
93+
resource "aws_s3_bucket" "mwaa_etl_scripts_bucket" {
94+
bucket = "${local.identifier_prefix}-mwaa-etl-scripts-bucket"
95+
96+
tags = module.tags.values
97+
}
98+
99+
resource "aws_s3_bucket_server_side_encryption_configuration" "mwaa_etl_scripts_bucket_encryption" {
100+
bucket = aws_s3_bucket.mwaa_etl_scripts_bucket.id
101+
102+
rule {
103+
apply_server_side_encryption_by_default {
104+
sse_algorithm = "aws:kms"
105+
kms_master_key_id = aws_kms_key.mwaa_key.arn
106+
}
107+
bucket_key_enabled = true
108+
}
109+
}
110+
111+
resource "aws_s3_bucket_public_access_block" "mwaa_etl_scripts_bucket_block" {
112+
bucket = aws_s3_bucket.mwaa_etl_scripts_bucket.id
113+
114+
block_public_acls = true
115+
block_public_policy = true
116+
ignore_public_acls = true
117+
restrict_public_buckets = true
118+
}

0 commit comments

Comments
 (0)