Skip to content

feat(fluentbit): adds support for workers to syslog output#1874

Open
mob-galtryn wants to merge 3 commits intofluent:masterfrom
mob-galtryn:adds-support-for-workers-syslog
Open

feat(fluentbit): adds support for workers to syslog output#1874
mob-galtryn wants to merge 3 commits intofluent:masterfrom
mob-galtryn:adds-support-for-workers-syslog

Conversation

@mob-galtryn
Copy link
Copy Markdown

Signed-off-by: Nick Galtry nick.galtry@mobilise.cloud

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #1870

Does this PR introduced a user-facing change?

Adds support for setting the number of workers for the syslog output.

Additional documentation, usage docs, etc.:


Comment thread apis/fluentbit/v1alpha2/plugins/output/syslog_types.go Outdated
@joshuabaird
Copy link
Copy Markdown
Collaborator

@mob-galtryn Would you mind taking a look at tests for other output plugins (eg, datadog, gelf, s3) and implement tests for the syslog output plugin in a similar fashion? Thanks!

@mob-galtryn
Copy link
Copy Markdown
Author

@joshuabaird I have found tests here apis/fluentbit/v1alpha2/clusteroutput_types_test.go that includes tests for the syslog output, are these the tests you are referring to?

@joshuabaird joshuabaird changed the title adds support for workers to syslog output fluentbit: adds support for workers to syslog output Feb 18, 2026
@joshuabaird joshuabaird changed the title fluentbit: adds support for workers to syslog output feat(fluentbit): adds support for workers to syslog output Feb 18, 2026
@joshuabaird joshuabaird added fluent-bit enhancement New feature or request labels Feb 18, 2026
@joshuabaird
Copy link
Copy Markdown
Collaborator

@joshuabaird I have found tests here apis/fluentbit/v1alpha2/clusteroutput_types_test.go that includes tests for the syslog output, are these the tests you are referring to?

Tests would probably go in a new syslog_types_test.go file.

@Cajga
Copy link
Copy Markdown
Contributor

Cajga commented Mar 19, 2026

@joshuabaird could you please review if the test is ok?

Signed-off-by: Nick Galtry <nick.galtry@mobilise.cloud>
Signed-off-by: Nick Galtry <nick.galtry@mobilise.cloud>
@marcofranssen marcofranssen force-pushed the adds-support-for-workers-syslog branch from 02696c4 to be41f22 Compare April 2, 2026 07:08
@marcofranssen
Copy link
Copy Markdown
Collaborator

@Cajga the CI is failing

Signed-off-by: Nick Galtry <nick.galtry@mobilise.cloud>
@wenchajun
Copy link
Copy Markdown
Member

please resolve the merge conflicts in this pull request

@marcofranssen
Copy link
Copy Markdown
Collaborator

@copilot resolve the merge conflicts in this pull request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request fluent-bit

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add option to configure workers for syslog output

5 participants