Skip to content

Commit 0d389d2

Browse files
Fix Do a dangling comma if alb_logs_bucket == "" (#50)
1 parent 12b6691 commit 0d389d2

File tree

1 file changed

+8
-7
lines changed

1 file changed

+8
-7
lines changed

src/resources/argocd-values.yaml.tpl

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -33,13 +33,14 @@ server:
3333
alb.ingress.kubernetes.io/backend-protocol: HTTPS
3434
alb.ingress.kubernetes.io/listen-ports: '[{"HTTP": 80},{"HTTPS":443}]'
3535
alb.ingress.kubernetes.io/ssl-redirect: '443'
36-
alb.ingress.kubernetes.io/load-balancer-attributes:
37-
routing.http.drop_invalid_header_fields.enabled=true,
38-
%{ if alb_logs_bucket != "" ~}
39-
access_logs.s3.enabled=true,
40-
access_logs.s3.bucket=${alb_logs_bucket},
41-
access_logs.s3.prefix=${alb_logs_prefix}
42-
%{ endif ~}
36+
alb.ingress.kubernetes.io/load-balancer-attributes: ${join(",", concat(
37+
["routing.http.drop_invalid_header_fields.enabled=true"],
38+
alb_logs_bucket != "" ? [
39+
"access_logs.s3.enabled=true",
40+
"access_logs.s3.bucket=${alb_logs_bucket}",
41+
"access_logs.s3.prefix=${alb_logs_prefix}"
42+
] : []
43+
))}
4344
%{ if forecastle_enabled == true ~}
4445
forecastle.stakater.com/appName: "ArgoCD"
4546
forecastle.stakater.com/expose: "true"

0 commit comments

Comments
 (0)