Skip to content

Conversation

github-actions[bot]
Copy link
Contributor

No description provided.

@donoghuc donoghuc force-pushed the update_release_notes_1752530221 branch from eb92ca6 to 3fbc3fe Compare July 24, 2025 18:49
@donoghuc donoghuc marked this pull request as ready for review July 24, 2025 18:50
Copy link
Contributor Author

github-actions bot commented Jul 24, 2025

🔍 Preview links for changed docs

@donoghuc donoghuc requested review from karenzone and robbavey July 24, 2025 18:50
@donoghuc
Copy link
Member

Updated to match pattern for rolling up previous minor patch releases into new minor release notes. #17823

Copy link
Member

@robbavey robbavey left a comment

Choose a reason for hiding this comment

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

Couple of suggestions


* Introduces `proxy` param to support proxy [#316](https://github.com/elastic/logstash-filter-elastic_integration/pull/316)
* Embeds Ingest Node components from Elasticsearch 9.1
* Introduces `proxy` config to support proxy URI to connect to Elasticsearch. [#320](https://github.com/elastic/logstash-filter-elastic_integration/pull/320)
Copy link
Member

Choose a reason for hiding this comment

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

Dupe of #316 above

Copy link
Member

Choose a reason for hiding this comment

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

oops! good catch

* Significantly improves write speeds to the persistent queue (PQ) when a pipeline's workers are caught up with already-written events [#17791](https://github.com/elastic/logstash/pull/17791)
* Eliminated log warning about unknown gauge metric type when using pipeline-to-pipeline. [#17721](https://github.com/elastic/logstash/pull/17721)
* Improve plugins remove command to support multiple plugins [#17030](https://github.com/elastic/logstash/pull/17030)
* Dropped the persistent queue setting `queue.checkpoint.interval`[#17759](https://github.com/elastic/logstash/pull/17759)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* Dropped the persistent queue setting `queue.checkpoint.interval`[#17759](https://github.com/elastic/logstash/pull/17759)
* Deprecated the persistent queue setting `queue.checkpoint.interval`[#17759](https://github.com/elastic/logstash/pull/17759), which was found to have no effect. This will be removed in a future Logstash release.

Let's soften the language here a little, as this is only a deprecation at the moment and note that the setting didn't actually do anything...

Copy link
Member

Choose a reason for hiding this comment

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

Will do.

@donoghuc donoghuc requested a review from robbavey July 25, 2025 18:57
Copy link
Member

@robbavey robbavey left a comment

Choose a reason for hiding this comment

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

We should add bullets

Co-authored-by: Rob Bavey <rob.bavey@elastic.co>
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.

Left some comments inline for consideration. Otherwise, LGTM.

General comment: We should align on tense for entries. In the first section alone, we have "improveS", "eliminateD", and "improve_" This is NOT blocking.


##### ES|QL Support in Elasticsearch Input (Technical Preview)

* Support for using ES|QL queries in the [Elasticsearch input](https://www.elastic.co/guide/en/logstash/current/plugins-inputs-elasticsearch.html) to add improved flexibility when ingesting data from Elasticsearch is now in Technical Preview.
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure that linking to the highest level of the Elasticsearch input adds value is such a specific piece of content. Suggest either deep linking to new docs, or removing the link.

And/or adding a link to the PR where the work was done.

Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>
@donoghuc donoghuc requested a review from robbavey July 28, 2025 18:43
Copy link
Member

@robbavey robbavey left a comment

Choose a reason for hiding this comment

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

Added a suggestion to follow up on @karenzone's suggestion to not link to ES Input front page for ES|QL support

Co-authored-by: Rob Bavey <rob.bavey@elastic.co>
Co-authored-by: Rob Bavey <rob.bavey@elastic.co>
@donoghuc donoghuc merged commit f63da8a into 9.1 Jul 28, 2025
9 of 10 checks passed
@donoghuc donoghuc deleted the update_release_notes_1752530221 branch July 28, 2025 23:09
@donoghuc
Copy link
Member

@Mergifyio backport main

Copy link
Contributor

mergify bot commented Jul 29, 2025

backport main

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Jul 29, 2025
* Update release notes for 9.1.0 * Human based release note edits * Roll up 9.0.1 -> 9.0.4 RN into 9.1.0 RN * code review feedback * Update docs/release-notes/index.md Co-authored-by: Rob Bavey <rob.bavey@elastic.co> * Update docs/release-notes/index.md Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com> * Update docs/release-notes/index.md Co-authored-by: Rob Bavey <rob.bavey@elastic.co> * Update docs/release-notes/index.md Co-authored-by: Rob Bavey <rob.bavey@elastic.co> --------- Co-authored-by: logstashmachine <43502315+logstashmachine@users.noreply.github.com> Co-authored-by: donoghuc <cas.donoghue@gmail.com> Co-authored-by: Rob Bavey <rob.bavey@elastic.co> Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com> (cherry picked from commit f63da8a)
donoghuc added a commit that referenced this pull request Jul 29, 2025
* Update release notes for 9.1.0 * Human based release note edits * Roll up 9.0.1 -> 9.0.4 RN into 9.1.0 RN * code review feedback * Update docs/release-notes/index.md * Update docs/release-notes/index.md * Update docs/release-notes/index.md * Update docs/release-notes/index.md --------- (cherry picked from commit f63da8a) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: logstashmachine <43502315+logstashmachine@users.noreply.github.com> Co-authored-by: donoghuc <cas.donoghue@gmail.com> Co-authored-by: Rob Bavey <rob.bavey@elastic.co> Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants