search

fluent / fluent-operator

Operate Fluent Bit and Fluentd in the Kubernetes way - Previously known as FluentBit Operator
Apache License 2.0
555 stars 229 forks source link

Fix fluentd path issues. #1195

Closed sarathchandra24 closed 3 weeks ago

sarathchandra24 commented 3 weeks ago

What this PR does / why we need it:

There has been an issue with fluentd coming up in docker images. Previous changes were handling only arch, as usr/local/bundle/bin/fluentd. But,

For amd64 it is"/usr/bin/fluentd" For arm64 it is "/usr/local/bundle/bin/fluentd".

Which issue(s) this PR fixes:

Fixes #1187

Does this PR introduced a user-facing change?

Additional documentation, usage docs, etc.:

benjaminhuo commented 3 weeks ago

That's great, thank you @sarathchandra24 !