Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Typo In aws-for-fluent-bit template firehose.extraOutputs section #1107

Open
sandidgm opened this issue May 22, 2024 · 0 comments · May be fixed by #1168
Open

Typo In aws-for-fluent-bit template firehose.extraOutputs section #1107

sandidgm opened this issue May 22, 2024 · 0 comments · May be fixed by #1168
Labels
bug Something isn't working

Comments

@sandidgm
Copy link

Describe the bug
typo in configmap.yaml template - in the .Values.firehose.enabled if block the extraOutputs uses kinesis and not firehose

Steps to reproduce

Expected outcome
to pass values to the firehose section I would like to use .Values.firehose.extraOutputs but I have to use .Values.kinesis.extraOutputs instead

Environment

  • Chart name: aws-for-fluent-bit
  • Chart version: v0.0.170
  • Kubernetes version: n/a
  • Using EKS (yes/no), if so version? n/a

Additional Context:

https://github.com/aws/eks-charts/blob/master/stable/aws-for-fluent-bit/templates/configmap.yaml#L187
@sandidgm sandidgm added the bug Something isn't working label May 22, 2024
@bryantbiggs bryantbiggs linked a pull request Oct 3, 2024 that will close this issue
Open
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore: Deprecate aws-for-fluent-bit chart in favor of publishing to Public ECR from upstream repo bryantbiggs/eks-charts
1 participant
@sandidgm