Skip to content

Commit

Permalink
fix: fluentbit use correct cw retention variable (#822)
Browse files Browse the repository at this point in the history
  • Loading branch information
Zvikan committed Aug 3, 2022
1 parent f7f1708 commit 7064fbb
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 10 deletions.
2 changes: 1 addition & 1 deletion docs/add-ons/aws-for-fluent-bit.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,14 +22,14 @@ You can optionally customize the Helm chart that deploys `aws_for_fluentbit` via
```hcl
enable_aws_for_fluentbit = true
aws_for_fluentbit_irsa_policies = ["IAM Policies"] # Add list of additional policies to IRSA to enable access to Kinesis, OpenSearch etc.
aws_for_fluentbit_cw_log_group_retention = 90
aws_for_fluentbit_helm_config = {
name = "aws-for-fluent-bit"
chart = "aws-for-fluent-bit"
repository = "https://aws.github.io/eks-charts"
version = "0.1.0"
namespace = "logging"
aws_for_fluent_bit_cw_log_group = "/${local.cluster_id}/worker-fluentbit-logs" # Optional
aws_for_fluentbit_cwlog_retention_in_days = 90
create_namespace = true
values = [templatefile("${path.module}/values.yaml", {
region = data.aws_region.current.name,
Expand Down
18 changes: 9 additions & 9 deletions examples/complete-kubernetes-addons/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -179,16 +179,16 @@ module "eks_blueprints_kubernetes_addons" {
enable_amazon_prometheus = true
amazon_prometheus_workspace_endpoint = module.managed_prometheus.workspace_prometheus_endpoint

enable_aws_for_fluentbit = true
enable_aws_for_fluentbit = true
aws_for_fluentbit_cw_log_group_retention = 30
aws_for_fluentbit_helm_config = {
name = "aws-for-fluent-bit"
chart = "aws-for-fluent-bit"
repository = "https://aws.github.io/eks-charts"
version = "0.1.16"
namespace = "logging"
aws_for_fluent_bit_cw_log_group = "/${module.eks_blueprints.eks_cluster_id}/worker-fluentbit-logs" # Optional
aws_for_fluentbit_cwlog_retention_in_days = 90
create_namespace = true
name = "aws-for-fluent-bit"
chart = "aws-for-fluent-bit"
repository = "https://aws.github.io/eks-charts"
version = "0.1.16"
namespace = "logging"
aws_for_fluent_bit_cw_log_group = "/${module.eks_blueprints.eks_cluster_id}/worker-fluentbit-logs" # Optional
create_namespace = true
values = [templatefile("${path.module}/helm_values/aws-for-fluentbit-values.yaml", {
region = local.region
aws_for_fluent_bit_cw_log_group = "/${module.eks_blueprints.eks_cluster_id}/worker-fluentbit-logs"
Expand Down

0 comments on commit 7064fbb

Please sign in to comment.