Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add suppress_type_name to stop warnings with ES7 #174

Merged
merged 1 commit into from
Nov 23, 2021

Conversation

bbezak
Copy link
Member

@bbezak bbezak commented Nov 23, 2021

When using elasticsearch 7 with fluentd, you seem to get a lot
of warnings in the docker logs output that look like:

[types removal] Specifying types in bulk requests is deprecated.

The docs suggest adding suppress_type_name to stop these warnings,
and that seems to work without affecting any functionality.

Further info here:
uken/fluent-plugin-elasticsearch#785

Closes-Bug: #1930856
Change-Id: I45be67df3717f78d78bcdc7df69600ab8681922f
(cherry picked from commit dee9d22)

When using elasticsearch 7 with fluentd, you seem to get a lot
of warnings in the docker logs output that look like:

    [types removal] Specifying types in bulk requests is deprecated.

The docs suggest adding suppress_type_name to stop these warnings,
and that seems to work without affecting any functionality.

Further info here:
uken/fluent-plugin-elasticsearch#785

Closes-Bug: #1930856
Change-Id: I45be67df3717f78d78bcdc7df69600ab8681922f
(cherry picked from commit dee9d22)
@bbezak
Copy link
Member Author

bbezak commented Nov 23, 2021

clean cherry-pick.
Backport needed as ES7 is used in stackhpc/victoria branch - upstream stable/victoria uses es6

@bbezak bbezak merged commit e75b7ae into stackhpc/victoria Nov 23, 2021
@markgoddard markgoddard deleted the stackhpc/victoria-stop-es7-warnings branch November 23, 2021 14:04
@cityofships cityofships added the monitoring All things related to observability & telemetry label Jan 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
monitoring All things related to observability & telemetry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants