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
@github-actions
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

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.

@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

@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

@mergify
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 <[email protected]>

* Update docs/release-notes/index.md

Co-authored-by: Karen Metts <[email protected]>

* Update docs/release-notes/index.md

Co-authored-by: Rob Bavey <[email protected]>

* Update docs/release-notes/index.md

Co-authored-by: Rob Bavey <[email protected]>

---------

Co-authored-by: logstashmachine <[email protected]>
Co-authored-by: donoghuc <[email protected]>
Co-authored-by: Rob Bavey <[email protected]>
Co-authored-by: Karen Metts <[email protected]>
(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 <[email protected]>
Co-authored-by: donoghuc <[email protected]>
Co-authored-by: Rob Bavey <[email protected]>
Co-authored-by: Karen Metts <[email protected]>
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.

4 participants