Opensearch Image Improvements

fluent / fluentd-kubernetes-daemonset

Fluentd daemonset for Kubernetes and it Docker image

Apache License 2.0

1.26k stars 978 forks source link

Opensearch Image Improvements #1390

Closed Magnitus- closed 1 year ago

Magnitus- commented 1 year ago

Added log stash options as environment variables
Added buffering options as environment variables

Let me know if you require any adjustments.

Magnitus- commented 1 year ago

I'll rework the commits tomorrow. Let me know if you want to keep version 1.0.6 of fluent-plugin-opensearch in the template/gemfile.erb or bump it up to 1.0.8. That part is a bit nebulous for me at the moment.

On our end, we have validated 1.0.8 of fluent-plugin-opensearch against the previous 1.15.2 release using self-published images (reading logs from 4 kubernetes clusters and sending them to an on-prem opensearch cluster running on version 2.2.1 of opensearch). We can try it against the updated 1.15.3 release if you like.

Magnitus- commented 1 year ago

I'll rework the commits tomorrow. Let me know if you want to keep version 1.0.6 of fluent-plugin-opensearch in the template/gemfile.erb or bump it up to 1.0.8. That part is a bit nebulous for me at the moment.

On our end, we have validated 1.0.8 of fluent-plugin-opensearch against the previous 1.15.2 release using self-published images (reading logs from 4 kubernetes clusters and sending them to an on-prem opensearch cluster running on version 2.2.1 of opensearch). We can try it against the updated 1.15.3 release if you like.

The comment above can be ignored. I hadn't read your reply. It is clear now.

Magnitus- commented 1 year ago

I made the adjustments. Let me know if you see something else.

ashie commented 1 year ago

@cosmo0920 Could you check whether adding these parameters is reasonable?

ashie commented 1 year ago

Thanks!