Skip to content

(dev) Add fluentbit to dev site. #940

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

Merged
merged 1 commit into from
Jul 8, 2025

Conversation

gseriche
Copy link
Contributor

@gseriche gseriche commented May 22, 2025

This pull request introduces significant updates to the Fluent Bit configuration and deployment files to improve logging, monitoring, and customization for Kubernetes clusters. The changes focus on removing outdated configurations, adding new filters and parsers, and enhancing Helm chart customizations for specific environments.

Removal of outdated configurations:

Enhanced Helm chart customizations:

Improved Fluent Bit configuration:

Monitoring improvements:

Pending PR: #934

@gseriche gseriche self-assigned this May 22, 2025
@gseriche gseriche added the enhancement New feature or request label May 22, 2025
@gseriche gseriche force-pushed the IT-6188_implement_fluentbit_on_kueyen branch 8 times, most recently from 930311a to d9835b7 Compare May 22, 2025 21:33
@gseriche gseriche force-pushed the IT-6188_implement_fluentbit_on_kueyen branch from d9835b7 to a6cc3e2 Compare June 5, 2025 16:45
@gseriche gseriche marked this pull request as ready for review June 5, 2025 16:45
@gseriche gseriche force-pushed the IT-6188_implement_fluentbit_on_kueyen branch 8 times, most recently from c966b91 to 828b1a9 Compare June 12, 2025 14:13
@gseriche gseriche force-pushed the IT-6188_implement_fluentbit_on_kueyen branch from 828b1a9 to 3040c79 Compare June 13, 2025 19:47
Copy link
Member

@jhoblitt jhoblitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure I understand the title of this PR. The bundle being modified isn't included by any cluster. Is the intent to deploy fluentbit on all dev clusters?

@gseriche gseriche force-pushed the IT-6188_implement_fluentbit_on_kueyen branch from 3040c79 to 687483a Compare June 18, 2025 21:46
@gseriche gseriche force-pushed the IT-6188_implement_fluentbit_on_kueyen branch from 687483a to bf2a059 Compare June 26, 2025 22:18
@gseriche
Copy link
Contributor Author

I'm not sure I understand the title of this PR. The bundle being modified isn't included by any cluster. Is the intent to deploy fluentbit on all dev clusters?

Yes, @jhoblitt, and next steps should be to add fluent-bit to the ls and cp.
Fluent-bit will extract the logs from the containers and send them to Loki.

@gseriche gseriche requested a review from jhoblitt June 26, 2025 22:19
@gseriche gseriche force-pushed the IT-6188_implement_fluentbit_on_kueyen branch 2 times, most recently from 541d241 to edd890d Compare July 3, 2025 23:55
Copy link
Contributor

@csilva-cl csilva-cl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it looks ok, but i want to confirm it was tested in dev. (I cant see the ns)

@csilva-cl
Copy link
Contributor

it looks ok, but i want to confirm it was tested in dev. (I cant see the ns)

scratch that, I can see the ns

@gseriche gseriche force-pushed the IT-6188_implement_fluentbit_on_kueyen branch from edd890d to d0ce1dd Compare July 4, 2025 13:55
@gseriche gseriche requested a review from csilva-cl July 4, 2025 14:15
@gseriche gseriche force-pushed the IT-6188_implement_fluentbit_on_kueyen branch from d0ce1dd to 2cc2866 Compare July 4, 2025 14:16
Copy link
Contributor

@cbarria cbarria left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@gseriche gseriche merged commit 711e818 into master Jul 8, 2025
14 checks passed
@gseriche gseriche deleted the IT-6188_implement_fluentbit_on_kueyen branch July 8, 2025 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants