Skip to content

Conversation

@jeanfabrice
Copy link
Contributor

Fixing default datastream name to logs-generic-default rather than logs-generic-logstash as data_stream_namespace defaults to default: https://github.com/logstash-plugins/logstash-output-elasticsearch/blob/main/lib/logstash/outputs/elasticsearch/data_stream_support.rb#L23

Thanks for contributing to Logstash! If you haven't already signed our CLA, here's a handy link: https://www.elastic.co/contributor-agreement/

Fixing default datastream name to `logs-generic-default` rather than `logs-generic-logstash` as `data_stream_namespace` defaults to `default`: https://github.com/logstash-plugins/logstash-output-elasticsearch/blob/main/lib/logstash/outputs/elasticsearch/data_stream_support.rb#L23
Copy link
Contributor

@karenzone karenzone left a comment

Choose a reason for hiding this comment

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

Docs look good.

Copy link
Contributor

@mashhurs mashhurs left a comment

Choose a reason for hiding this comment

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

Thanks for the fix! 💯

@karenzone
Copy link
Contributor

@logstashmachine bump patch

Copy link
Contributor

@karenzone karenzone left a comment

Choose a reason for hiding this comment

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

Content correct and conflicts resolved

@karenzone karenzone merged commit 81de72e into logstash-plugins:main Apr 1, 2024
@karenzone
Copy link
Contributor

Published to rubygems.org at v.11.22.5

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