Skip to content

Commit efcd4f5

Browse files
authored
Merge pull request #23 from Optum/Fix/ModuleFile
Move to module path for file object
2 parents 3b82ef8 + 737484e commit efcd4f5

2 files changed

Lines changed: 8 additions & 3 deletions

File tree

modules/artifacts_bucket.tf

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,8 @@
1+
locals {
2+
redbox_principal_policy = var.redbox_principal_policy == "" ? "${path.module}/fixtures/policies/redbox_principal_policy.tmpl" : var.redbox_principal_policy
3+
}
4+
5+
16
# Configure an S3 Bucket to hold artifacts
27
# (eg. application code deployments, etc.)
38
resource "aws_s3_bucket" "artifacts" {
@@ -62,6 +67,6 @@ POLICY
6267
resource "aws_s3_bucket_object" "redbox_principal_policy" {
6368
bucket = aws_s3_bucket.artifacts.id
6469
key = "fixtures/policies/redbox_principal_policy.tmpl"
65-
source = var.redbox_principal_policy
66-
etag = "${filemd5(var.redbox_principal_policy)}"
70+
source = local.redbox_principal_policy
71+
etag = "${filemd5(local.redbox_principal_policy)}"
6772
}

modules/variables.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -163,5 +163,5 @@ variable "budget_notification_threshold_percentiles" {
163163
variable "redbox_principal_policy" {
164164
type = string
165165
description = "Location of file with the policy used for the RedBox Principal Account"
166-
default = "fixtures/policies/redbox_principal_policy.tmpl"
166+
default = ""
167167
}

0 commit comments

Comments
 (0)