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

Update 8.13.0 queue documentation and release notes #983

Merged
merged 4 commits into from
Mar 27, 2024

Conversation

cmacknz
Copy link
Member

@cmacknz cmacknz commented Mar 26, 2024

Bring over two relevant output changes from Beats.

Also sync the queue.flush.min_events documentation from Beats so that it is up to date with the most recent changes.

@cmacknz cmacknz requested review from faec and kilfoyle March 26, 2024 20:23
@cmacknz cmacknz self-assigned this Mar 26, 2024
@cmacknz cmacknz requested a review from a team as a code owner March 26, 2024 20:23
Copy link

A documentation preview will be available soon.

Request a new doc build by commenting
  • Rebuild this PR: run docs-build
  • Rebuild this PR and all Elastic docs: run docs-build rebuild

run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.

If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here.

@cmacknz
Copy link
Member Author

cmacknz commented Mar 26, 2024

The changes here are ported over from https://github.com/elastic/beats/pull/37795/files

@cmacknz
Copy link
Member Author

cmacknz commented Mar 26, 2024

The bulk_max_size docs seem to not be up to date either here or in Beats either https://github.com/elastic/beats/blob/main/libbeat/outputs/elasticsearch/docs/elasticsearch.asciidoc#bulk_max_size

@cmacknz
Copy link
Member Author

cmacknz commented Mar 26, 2024

Fixing the bulk_max_size docs in beats: elastic/beats#38634

kilfoyle
kilfoyle previously approved these changes Mar 26, 2024
Copy link
Contributor

@kilfoyle kilfoyle left a comment

Choose a reason for hiding this comment

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

Just the one suggestion but otherwise LGTM!

@cmacknz cmacknz merged commit ac080b9 into elastic:main Mar 27, 2024
3 checks passed
@cmacknz cmacknz deleted the update-8.13.0-queue-docs branch March 27, 2024 13:01
mergify bot pushed a commit that referenced this pull request Mar 27, 2024
* Port output changes from Beats.

* Sync queue docs from Beats.

* Update docs/en/ingest-management/elastic-agent/configuration/outputs/output-logstash.asciidoc

Co-authored-by: David Kilfoyle <[email protected]>

* Add backticks to other outputs.

---------

Co-authored-by: David Kilfoyle <[email protected]>
(cherry picked from commit ac080b9)
cmacknz added a commit that referenced this pull request Mar 27, 2024
* Port output changes from Beats.

* Sync queue docs from Beats.

* Update docs/en/ingest-management/elastic-agent/configuration/outputs/output-logstash.asciidoc

Co-authored-by: David Kilfoyle <[email protected]>

* Add backticks to other outputs.

---------

Co-authored-by: David Kilfoyle <[email protected]>
(cherry picked from commit ac080b9)

Co-authored-by: Craig MacKenzie <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants