Skip to content

Commit cf0cc18

Browse files
authored
fix: issue #304 layer2-aws terraform plan error (#305)
* fix: issue #304 layer2-aws terraform plan error * issue #305 fix tfsec * issue #305 fix tfsec * issue #305 fix tfsec
1 parent 917000c commit cf0cc18

File tree

3 files changed

+5
-2
lines changed

3 files changed

+5
-2
lines changed

terraform/layer1-aws/aws-cloudtrail.tf

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
#tfsec:ignore:aws-cloudtrail-enable-at-rest-encryption
1+
#tfsec:ignore:aws-cloudtrail-enable-at-rest-encryption tfsec:ignore:aws-cloudtrail-ensure-cloudwatch-integration
22
resource "aws_cloudtrail" "main" {
33
name = local.name
44
s3_bucket_name = aws_s3_bucket.cloudtrail.id
@@ -10,7 +10,7 @@ resource "aws_cloudtrail" "main" {
1010
tags = local.tags
1111
}
1212

13-
#tfsec:ignore:aws-s3-enable-bucket-logging tfsec:ignore:aws-s3-enable-versioning
13+
#tfsec:ignore:aws-s3-enable-bucket-logging tfsec:ignore:aws-s3-enable-versioning tfsec:ignore:aws-cloudtrail-require-bucket-access-logging
1414
resource "aws_s3_bucket" "cloudtrail" {
1515
bucket = "${local.name}-aws-cloudtrail-logs"
1616

terraform/layer1-aws/aws-vpc.tf

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ data "aws_security_group" "default" {
1313
vpc_id = module.vpc.vpc_id
1414
}
1515

16+
#tfsec:ignore:aws-ec2-no-public-ip-subnet
1617
module "vpc" {
1718
source = "terraform-aws-modules/vpc/aws"
1819
version = "3.12.0"

terraform/layer2-k8s/eks-gitlab-runner.tf

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -144,6 +144,8 @@ resource "aws_s3_bucket_server_side_encryption_configuration" "gitlab_runner_enc
144144
}
145145

146146
resource "aws_s3_bucket_lifecycle_configuration" "gitlab_runner_lifecycle" {
147+
count = local.gitlab_runner.enabled ? 1 : 0
148+
147149
bucket = aws_s3_bucket.gitlab_runner_cache[0].id
148150

149151
rule {

0 commit comments

Comments
 (0)