Skip to content

Conversation

github-actions[bot]
Copy link
Contributor

No description provided.

@github-actions
Copy link
Contributor Author

github-actions bot commented Oct 15, 2025

🔍 Preview links for changed docs

@donoghuc donoghuc marked this pull request as ready for review October 17, 2025 19:37

**Elastic_integration Filter - 9.2.0**

404: Not Found
Copy link
Member

Choose a reason for hiding this comment

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

👀

Copy link
Member

Choose a reason for hiding this comment

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

Proposed fix #18315

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.

I'd like to see a little more detail in a few of these areas

* Logging improvement while handling exceptions in the pipeline, ensuring that chained exceptions propagate enough information to be actionable. [#17935](https://github.com/elastic/logstash/pull/17935)
* [Support for using ES|QL queries](https://github.com/logstash-plugins/logstash-filter-elasticsearch/pull/194) in the Elasticsearch filter to add improved flexibility when ingesting data from Elasticsearch is now in Technical Preview.
* Gauge type metrics, such as current and peak connection counts of Elastic Agent, are now available in the `_node/stats` API response when the `vertices=true` parameter is included. These metrics are particularly useful for monitoring {{ls}} plugin activity on the {{ls}} Integration dashboards [#18090](https://github.com/elastic/logstash/pull/18090)
* Improve logstash release artifacts file metadata: mtime is preserved when buiilding tar archives [#18091](https://github.com/elastic/logstash/pull/18091)
Copy link
Member

Choose a reason for hiding this comment

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

Maybe we could add some explanation as to what this means (no more epoch timestamps for artifacts)

Copy link
Member

Choose a reason for hiding this comment

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

I could, but one thing to note here is that this is a copied RN from a 9.1.z patch release. If we do this we would probably want to do it everywhere?

Copy link
Member

Choose a reason for hiding this comment

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

Ok - let's keep it. This is probably sufficient information for folks who have come across this issue to understand it


##### Event compression for persisted queue

* Support for event compression in the persisted queue, controlled by the per-pipeline `queue.compression` setting, which defaults to `none` [#18121](https://github.com/elastic/logstash/pull/18121)
Copy link
Member

Choose a reason for hiding this comment

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

@yaauie Can you add detail here, and add color about the (backwards compatible) reduction in queue size without compression, and the new compression capabilities (and restrictions)

Copy link
Member

Choose a reason for hiding this comment

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

This is completed 6759207 Rye provided the copy for the content of that commit.


**Elastic_integration Filter - 9.2.0**

404: Not Found
Copy link
Member

Choose a reason for hiding this comment

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

I think we might need a few more details here... ;)

Copy link
Member

Choose a reason for hiding this comment

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

yep, I proposed a bugfix #18315 and updated with the output based on running that script with the proposed patch.

donoghuc and others added 2 commits October 17, 2025 13:07
Add changes based on generated message with #18315

### Known issues

* The `decode_size_limit_bytes` setting for plugins that use the `json_lines` codec behaves differently in 9.2.0. Specifically when the size limit exceeds the limit without a separator in the data the size will grow beyond the limit. The details for this issue and the details for the future behavior are being tracked in [#18321](https://github.com/elastic/logstash/issues/18321)
Copy link
Member

Choose a reason for hiding this comment

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

Should we add advice along the lines that this an expert-level setting you should not set unless you have specific needs that require it? cc @jsvd @yaauie

Copy link
Contributor

Choose a reason for hiding this comment

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

I think adding expert level note to the plugin doc makes more sense.

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.

LGTM (assuming others are ok with my changes here!)

Copy link
Contributor

@mashhurs mashhurs left a comment

Choose a reason for hiding this comment

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

Love this! LGTM!

@mashhurs mashhurs merged commit 2dea8c9 into 9.2 Oct 22, 2025
10 checks passed
@mashhurs mashhurs deleted the update_release_notes_1760570084 branch October 22, 2025 23:25
@mashhurs
Copy link
Contributor

@Mergifyio backport main

@mergify
Copy link
Contributor

mergify bot commented Oct 22, 2025

backport main

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Oct 22, 2025
* Update release notes for 9.2.0

* identify potential RN candidate commits

* Finalize release notes

* Restore changes for elastic_integration

Add changes based on generated message with #18315

* Update docs/release-notes/index.md

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

* Update docs/release-notes/index.md

Co-authored-by: Rye Biesemeyer <[email protected]>

* Update docs/release-notes/index.md

* update release highlight for pq improvements

* document buffered tokenizer change in behavior

* Update index.md with description of batch size metrics

Added batch size metrics info

* Updated Release Notes to add Highlights and Other Features and Enhancement Sections

---------

Co-authored-by: logstashmachine <[email protected]>
Co-authored-by: donoghuc <[email protected]>
Co-authored-by: Rob Bavey <[email protected]>
Co-authored-by: Rye Biesemeyer <[email protected]>
(cherry picked from commit 2dea8c9)
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.

5 participants