Skip to content
This repository was archived by the owner on Jan 25, 2023. It is now read-only.

Addresses an issue with logging configuration with improper for_each iterator #242

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Addresses an issue with logging configuration with improper for_each iterator #242

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
3ee2f5d
[n/a] revert the once
dchernivetsky Apr 12, 2021
58427ef
[n/a] use can for proper bool
dchernivetsky Apr 12, 2021
3fdd8d8
[n/a] missing paren
dchernivetsky Apr 12, 2021
281d73b
[n/a] remove use of can
dchernivetsky May 24, 2021
a73e7bd
Update modules/vault-elb/main.tf
dchernivetsky May 26, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Update modules/vault-elb/main.tf 
simplify the logic of enabling logs

Co-authored-by: Yevgeniy Brikman <[email protected]>
  • Loading branch information
@dchernivetsky @brikis98
dchernivetsky and brikis98 authored May 26, 2021
commit a73e7bdd1ebd85dc66f2fc9cd58f4469dace4c05
2 changes: 1 addition & 1 deletion modules/vault-elb/main.tf
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ resource "aws_elb" "vault" {
content {
# enabled flag is optional. The user can choose to specify or not. If not specified we check if bucket param is specified and assume that
# the user wants the logging to be enabled.
enabled = lookup(access_logs.value, "enabled", false) ? lookup(access_logs.value, "enabled") : can(lookup(access_logs.value, "bucket"))
enabled = lookup(access_logs.value, "enabled", true)
bucket = lookup(access_logs.value, "bucket", null)
bucket_prefix = lookup(access_logs.value, "bucket_prefix", null)
interval = lookup(access_logs.value, "interval", null)
Expand Down