Skip to content

Commit

Permalink
fix workflow failure issue
Browse files Browse the repository at this point in the history
  • Loading branch information
Maboh Christopher authored and Maboh Christopher committed Jun 14, 2024
1 parent e245395 commit 7f8d051
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 10 deletions.
4 changes: 1 addition & 3 deletions terraform/modules/bucket/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -86,11 +86,9 @@ resource "aws_s3_bucket_server_side_encryption_configuration" "this" {
}
}
}

data "aws_s3_bucket" "access_logs" {
bucket = "${var.app}-access-logs"
bucket = var.access_log_bucket_name
}

resource "aws_s3_bucket_logging" "this" {
bucket = aws_s3_bucket.this.id

Expand Down
11 changes: 4 additions & 7 deletions terraform/modules/bucket/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -9,11 +9,8 @@ variable "cross_account_read_roles" {
default = []
}

variable "app" {
description = "The application name (bcda, dpc)"
type = string
validation {
condition = contains(["bcda", "dpc"], var.app)
error_message = "Valid value for app is bcda, or dpc."
}
variable "access_log_bucket_name" {
type = string
description = "The name of the centralized access log bucket"
default = "access-logs"
}

0 comments on commit 7f8d051

Please sign in to comment.