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

ElasticSearchDestination: Display deprecated warning message about us… #2274

Merged

Conversation

pzoleex
Copy link
Collaborator

@pzoleex pzoleex commented Sep 12, 2018

…ing V1.x

The EOL date of the latest 1.7 Elastic is 2017.01.16.
Because it is not supported by the producer anymore, it is safe to remove from syslog-ng as well.
But before removing, syslog-ng will display a warning message about it.

…ing V1.x

The EOL date of the latest 1.7 Elastic is 2017.01.16.
Because it is not supported by the producer anymore, it is safe to remove from syslog-ng as well.
But before removing, syslog-ng will display a warning message about it.
Copy link
Collaborator

@Kokan Kokan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is out of scope, and just a note:
The grammar parsing could handle deprecated keywords, which cannot be used here because the elasticsearch is actually an scl. Maybe it would worth to introduce deprecation possibility on configuration level.

@kira-syslogng
Copy link
Contributor

Build SUCCESS

@szemere szemere merged commit 9763a35 into syslog-ng:master Sep 13, 2018
@lbudai lbudai added this to the OSE 3.18 milestone Sep 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants