Skip to content

Commit 3c24ddf

Browse files
committed
NRL-1188 Reference existing bucket
1 parent fa3d2c5 commit 3c24ddf

File tree

4 files changed

+5
-6
lines changed

4 files changed

+5
-6
lines changed

terraform/infrastructure/firehose.tf

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,4 @@ module "firehose__processor" {
88
splunk_environment = local.splunk_environment
99
splunk_index = local.splunk_index
1010
destination = "splunk"
11-
reporting_bucket = module.dev-glue.source_bucket_name
1211
}

terraform/infrastructure/modules/firehose/kinesis.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ resource "aws_kinesis_firehose_delivery_stream" "reporting_stream" {
6363

6464
extended_s3_configuration {
6565
role_arn = aws_iam_role.firehose.arn
66-
bucket_arn = var.reporting_bucket
66+
bucket_arn = data.aws_s3_bucket.source-data-bucket.arn
6767

6868
processing_configuration {
6969
enabled = "true"

terraform/infrastructure/modules/firehose/s3.tf

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -106,3 +106,7 @@ resource "aws_iam_policy" "firehose-alert--s3-read" {
106106
]
107107
})
108108
}
109+
110+
data "aws_s3_bucket" "source-data-bucket" {
111+
bucket = "${var.prefix}-source-data-bucket"
112+
}

terraform/infrastructure/modules/firehose/vars.tf

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,3 @@ variable "error_prefix" {
3434
type = string
3535
default = "errors"
3636
}
37-
38-
variable "reporting_bucket" {
39-
type = string
40-
}

0 commit comments

Comments
 (0)