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.

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: fluent-bit K8s enrichment #194

Merged
merged 1 commit into from
Dec 23, 2024
Merged

fix: fluent-bit K8s enrichment #194

merged 1 commit into from
Dec 23, 2024

Conversation

lemaitre-aneo
Copy link
Contributor

@lemaitre-aneo lemaitre-aneo commented Dec 18, 2024

Motivation

Log enrichment for kubernetes metadata was not working when fluent-bit was configured on daemon-set.

Description

Configure fluent-bit to fetch kubernetes metadata from Kupe-API instead of kubelet, which was not working.
It also remove the unused volume mount on the sidecar that was failing deployment on hosts without /var/fluent-bit/state/

Testing

Tested on locahost with K3s and docker.

Impact

Apart from fixing the bug, there is no impact.

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have performed a self-review of my code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • I have thoroughly tested my modifications and added tests when necessary.
  • Tests pass locally and in the CI.
  • I have assessed the performance impact of my modifications.

@lemaitre-aneo lemaitre-aneo merged commit 25e9468 into main Dec 23, 2024
5 checks passed
@lemaitre-aneo lemaitre-aneo deleted the fl/k8s-fluentbit branch December 23, 2024 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants