Skip to content

update Beats capability for DStreams#1964

Merged
dedemorton merged 1 commit into8.3from
hythloday-zero-beats-cap-dstreams
Jun 29, 2022
Merged

update Beats capability for DStreams#1964
dedemorton merged 1 commit into8.3from
hythloday-zero-beats-cap-dstreams

Conversation

@Hythloday-zero
Copy link
Copy Markdown
Contributor

As of Beats 8.0, data streams are the default output to Elasticsearch. Line 245 needs change to {y} and the phrasing in Line 248 should reflect that all three options can take advantage of data streams, easier ILM, and the data stream naming scheme.

Reference: elastic/beats#28450 linked in https://www.elastic.co/guide/en/beats/libbeat/8.0/release-notes-8.0.0.html

As of Beats 8.0, data streams are the default output to Elasticsearch.  Line 245 needs change to {y} and the phrasing in Line 248 should reflect that all three options can take advantage of data streams, easier ILM, and the data stream naming scheme.

Reference: elastic/beats#28450 linked in https://www.elastic.co/guide/en/beats/libbeat/8.0/release-notes-8.0.0.html
@ghost
Copy link
Copy Markdown

ghost commented Jun 29, 2022

A documentation preview will be available soon:

Copy link
Copy Markdown
Contributor

@dedemorton dedemorton left a comment

Choose a reason for hiding this comment

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

Good catch! LGTM. This PR will need to be forward ported to main, and should be backported to 8.0-8.2. Do you want me to take care of that for you?

@Hythloday-zero
Copy link
Copy Markdown
Contributor Author

Good catch! LGTM. This PR will need to be forward ported to main, and should be backported to 8.0-8.2. Do you want me to take care of that for you?

Yes, thank you, that would be great- I wasn't sure how the process worked; I'm over on the Consulting side in AMER-FED and happened across this doing some research for a client.

@dedemorton
Copy link
Copy Markdown
Contributor

I wasn't sure how the process worked

This is great, but next time, select master from the version selector before you click Edit. Just makes it a bit easier to backport. No worries, though. It's good to have the content change.

@dedemorton dedemorton merged commit 3c301ea into 8.3 Jun 29, 2022
@dedemorton dedemorton deleted the hythloday-zero-beats-cap-dstreams branch June 29, 2022 23:07
dedemorton pushed a commit to dedemorton/observability-docs that referenced this pull request Jun 29, 2022
As of Beats 8.0, data streams are the default output to Elasticsearch.  Line 245 needs change to {y} and the phrasing in Line 248 should reflect that all three options can take advantage of data streams, easier ILM, and the data stream naming scheme.

Reference: elastic/beats#28450 linked in https://www.elastic.co/guide/en/beats/libbeat/8.0/release-notes-8.0.0.html
@Hythloday-zero
Copy link
Copy Markdown
Contributor Author

but next time, select master from the version selector before you click Edit.

Understood. ezpz

dedemorton added a commit that referenced this pull request Jun 30, 2022
Co-authored-by: Hythloday-zero <74616148+Hythloday-zero@users.noreply.github.com>
mergify bot pushed a commit that referenced this pull request Jun 30, 2022
Co-authored-by: Hythloday-zero <74616148+Hythloday-zero@users.noreply.github.com>
(cherry picked from commit 138880b)
mergify bot pushed a commit that referenced this pull request Jun 30, 2022
Co-authored-by: Hythloday-zero <74616148+Hythloday-zero@users.noreply.github.com>
(cherry picked from commit 138880b)
mergify bot pushed a commit that referenced this pull request Jun 30, 2022
Co-authored-by: Hythloday-zero <74616148+Hythloday-zero@users.noreply.github.com>
(cherry picked from commit 138880b)
dedemorton added a commit that referenced this pull request Jun 30, 2022
Co-authored-by: Hythloday-zero <74616148+Hythloday-zero@users.noreply.github.com>
(cherry picked from commit 138880b)

Co-authored-by: DeDe Morton <dede.morton@elastic.co>
dedemorton added a commit that referenced this pull request Jun 30, 2022
Co-authored-by: Hythloday-zero <74616148+Hythloday-zero@users.noreply.github.com>
(cherry picked from commit 138880b)

Co-authored-by: DeDe Morton <dede.morton@elastic.co>
dedemorton added a commit that referenced this pull request Jun 30, 2022
Co-authored-by: Hythloday-zero <74616148+Hythloday-zero@users.noreply.github.com>
(cherry picked from commit 138880b)

Co-authored-by: DeDe Morton <dede.morton@elastic.co>
bmorelli25 pushed a commit to bmorelli25/observability-docs that referenced this pull request Apr 11, 2023
Co-authored-by: Hythloday-zero <74616148+Hythloday-zero@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants