- Notifications
You must be signed in to change notification settings - Fork 3.5k
Release notes for 9.1.0 #17822
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
Release notes for 9.1.0 #17822
Conversation
eb92ca6
to 3fbc3fe
Compare 🔍 Preview links for changed docs |
Updated to match pattern for rolling up previous minor patch releases into new minor release notes. #17823 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple of suggestions
docs/release-notes/index.md Outdated
| ||
* 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) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dupe of #316 above
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oops! good catch
docs/release-notes/index.md Outdated
* 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) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* 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...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will do.
There was a problem hiding this 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>
There was a problem hiding this 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.
docs/release-notes/index.md Outdated
| ||
##### 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. |
There was a problem hiding this comment.
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>
There was a problem hiding this 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>
@Mergifyio backport main |
✅ Backports have been created
|
* 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)
* 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>
No description provided.