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

Add fluent-bit service and option to extend the RBAC configurations #462

Merged
merged 1 commit into from
Nov 23, 2022

Conversation

Kristian-ZH
Copy link
Member

Signed-off-by: Kristian-ZH k.zhelyazkov@sap.com

What this PR does / why we need it:

The PR adds fluent-bit service that can be disabled with the configuration DisableService. It also adds an option to extend the RBAC configurations through the FB/Fluentd Spec config file.

Motivation:
The Service is needed in case you want to easily scrape metrics from the fluent-bit from Prometheus for example.
The RBAC extension is needed in case you deploy your custom output plugin (As the operator and the FB itself already support this option) and this custom plugin works with additional resources.

Which issue(s) this PR fixes:

Fixes #

Does this PR introduced a user-facing change?

The operator now deploys fluent-bit service which can be disabled with `DisableService` configuration in the FluentBit
FluentBit and FluentD resources now support `rbacRules` configuration which allows to extend their RBACs

Signed-off-by: Kristian-ZH <k.zhelyazkov@sap.com>
@benjaminhuo
Copy link
Member

@Kristian-ZH This is a significant enhancement, thanks!
@wenchajun please pick this pr into the upcoming 1.7 release

@benjaminhuo benjaminhuo merged commit 84489b4 into fluent:master Nov 23, 2022
@benjaminhuo benjaminhuo mentioned this pull request Nov 23, 2022
@Kristian-ZH Kristian-ZH deleted the add-rbac-and-fb-service branch November 23, 2022 11:25
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.

None yet

2 participants