-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Correct fargate-fluentbit
example usage
#1131
Conversation
@@ -96,35 +96,6 @@ module "eks_blueprints" { | |||
} | |||
} | |||
|
|||
self_managed_node_groups = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not necessary to have all 3 compute types here for "complete addons"
@@ -159,13 +130,6 @@ module "eks_blueprints_kubernetes_addons" { | |||
enable_aws_for_fluentbit = true | |||
aws_for_fluentbit_cw_log_group_retention = 30 | |||
aws_for_fluentbit_helm_config = { | |||
name = "aws-for-fluent-bit" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just showing the bits that are custom
@@ -178,41 +142,6 @@ module "eks_blueprints_kubernetes_addons" { | |||
] | |||
} | |||
|
|||
enable_fargate_fluentbit = true | |||
fargate_fluentbit_addon_config = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is nearly (99.9%) identical to the defaults so removing entirely. usage of this addon is still shown though in the fargate-serverless
example
@@ -268,12 +197,3 @@ module "vpc" { | |||
|
|||
tags = local.tags | |||
} | |||
|
|||
data "aws_ami" "eks" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
was used on self-managed node group thats removed
@@ -121,6 +121,9 @@ module "eks_blueprints_kubernetes_addons" { | |||
|
|||
# Enable Fargate logging | |||
enable_fargate_fluentbit = true | |||
fargate_fluentbit_addon_config = { | |||
flb_log_cw = true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
from recent PR #1129
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm!
What does this PR do?
complete-kubernetes-addons
fargate-fluentbit
example usage; this is shown in thefargate-serverless
exampleMotivation
More
pre-commit run -a
with this PRNote: Not all the PRs require a new example and/or doc page. In general:
docs/add-ons/*
is required for new a new addonFor Moderators
Additional Notes