Skip to content

Commit dd39b07

Browse files
authored
Do not create data resources when disabled (#27)
1 parent ea43d1a commit dd39b07

File tree

1 file changed

+12
-4
lines changed

1 file changed

+12
-4
lines changed

main.tf

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,8 @@
1010
# Data
1111
#--------------------------------------------------------------
1212
data "aws_iam_policy_document" "assume_role" {
13+
count = local.enabled ? 1 : 0
14+
1315
statement {
1416
actions = ["sts:AssumeRole"]
1517
effect = "Allow"
@@ -22,6 +24,8 @@ data "aws_iam_policy_document" "assume_role" {
2224
}
2325

2426
data "aws_iam_policy_document" "es_logs" {
27+
count = local.enabled ? 1 : 0
28+
2529
statement {
2630
actions = [
2731
"logs:CreateLogGroup",
@@ -54,6 +58,8 @@ data "aws_iam_policy_document" "es_logs" {
5458
}
5559

5660
data "aws_iam_policy_document" "sns" {
61+
count = local.enabled ? 1 : 0
62+
5763
statement {
5864
actions = [
5965
"sns:Publish"
@@ -68,8 +74,10 @@ data "aws_iam_policy_document" "sns" {
6874
}
6975

7076
data "aws_iam_policy_document" "default" {
71-
source_json = data.aws_iam_policy_document.es_logs.json
72-
override_json = length(var.sns_arn) > 0 ? data.aws_iam_policy_document.sns.json : "{}"
77+
count = local.enabled ? 1 : 0
78+
79+
source_json = join("", data.aws_iam_policy_document.es_logs.*.json)
80+
override_json = length(var.sns_arn) > 0 ? join("", data.aws_iam_policy_document.sns.*.json) : "{}"
7381
}
7482

7583
locals {
@@ -189,15 +197,15 @@ resource "aws_security_group_rule" "ingress_to_es_cluster_from_lambda" {
189197
resource "aws_iam_role" "default" {
190198
count = local.enabled ? 1 : 0
191199
name = local.function_name
192-
assume_role_policy = data.aws_iam_policy_document.assume_role.json
200+
assume_role_policy = join("", data.aws_iam_policy_document.assume_role.*.json)
193201
tags = module.label.tags
194202
}
195203

196204
resource "aws_iam_role_policy" "default" {
197205
count = local.enabled ? 1 : 0
198206
name = local.function_name
199207
role = join("", aws_iam_role.default.*.name)
200-
policy = data.aws_iam_policy_document.default.json
208+
policy = join("", data.aws_iam_policy_document.default.*.json)
201209
}
202210

203211
resource "aws_iam_role_policy_attachment" "default" {

0 commit comments

Comments
 (0)