search

fluent / fluentd-kubernetes-daemonset

Fluentd daemonset for Kubernetes and it Docker image
Apache License 2.0
1.27k stars 980 forks source link

Expose more options to OpenSearch image #1493

Closed RemcodM closed 1 month ago

RemcodM commented 5 months ago

I will give this a second try after #1466 got closed because of inactivity. I have rebased and updated this PR accordingly.

This PR exposes some more options of the fluent-plugin-opensearch plug-in as environment variables to the Dockerfile, to be more in line with what the Elasticsearch images offer. This should make the transition from Elasticsearch to OpenSearch easier.

It only changes the fluentd.conf file for OpenSearch images. The fluentd.conf is generated using the Makefile, so it based on the template change included in this PR.

Please let me know if any changes are necessary to get this merged.

RemcodM commented 5 months ago

Could you modify only templates/conf/fluent.conf.erb? Files under docker-image/ are genegeted automatically.

Done!

github-actions[bot] commented 2 months ago

This PR has been automatically marked as stale because it has been open 90 days with no activity. Remove stale label or comment or this PR will be closed in 30 days

github-actions[bot] commented 1 month ago

This PR was automatically closed because of stale in 30 days