feat(fluentbit): adds support for workers to syslog output#1874
feat(fluentbit): adds support for workers to syslog output#1874mob-galtryn wants to merge 3 commits intofluent:masterfrom
Conversation
|
@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! |
|
@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 |
|
@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>
02696c4 to
be41f22
Compare
|
@Cajga the CI is failing |
Signed-off-by: Nick Galtry <nick.galtry@mobilise.cloud>
|
please resolve the merge conflicts in this pull request |
|
@copilot resolve the merge conflicts in this pull request |
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?
Additional documentation, usage docs, etc.: