-
Notifications
You must be signed in to change notification settings - Fork 13
Version bump to 8.17.1 and adds a changelog for PR #255. #260
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
Conversation
💚 Build Succeeded
|
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.
LGTM, left a doubt on the the release notes.
## 8.17.1 | ||
- Provides a guidance in logs when plugin version mismatches with connected Elasticsearch `major.minor` version [#255](https://github.com/elastic/logstash-filter-elastic_integration/pull/255) | ||
- Embeds Ingest Node components from Elasticsearch 8.17 | ||
- Compatible with Logstash 8.15+ |
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.
Given that 8.17.0 is already 8.15+ compatible should we repeat also in this patch version?
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 recently make plugin major.minor
alignment with stack versions and I support repeating this important baseline (helps users which ES version to connect to, and tells LS version compatibility) to make very visible for users.
* Version bump to 8.16.1 and and changelogs for PR elastic#253. (elastic#259) * Version bump to 8.16.1 and and changelogs for PR elastic#253. * Version bump to 8.17.1 and adds a changelog for PR elastic#255. (elastic#260) (cherry picked from commit 1b80fde) (cherry picked from commit c7a9dbd)
Prepares for 8.17.1 release.
Author's checklist:
8.x
main