-
Notifications
You must be signed in to change notification settings - Fork 3.5k
Release notes 7.3.0 #10993
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 7.3.0 #10993
Conversation
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.
Builds cleanly. One minor commit and two comments.
@@ -3,6 +3,8 @@ | |||
|
|||
This section summarizes the changes in the following releases: | |||
|
|||
* <<logstash-7-3-0,Logstash 7.3.0>> | |||
* <<logstash-7-2-0,Logstash 7.2.0>> |
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.
Do we need 7.2.1?
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.
🤔 I honestly wouldn't expect any of the patch-level releases to be included in a changelog for a minor-level release, and am a bit surprised to see them listed here. The changelog entry below for 7.3.0 includes fixes that also target a patch-level release on 7.2 (which hasn't been released), so I don't think there is risk in omitting it.
|
||
* Adding oid_path_length config option https://github.com/logstash-plugins/logstash-input-snmp/pull/59[#59] | ||
* Fixing bug with table support removing index value from OIDs https://github.com/logstash-plugins/logstash-input-snmp/issues/60[#60] | ||
|
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.
Should this issue be closed? It looks at least partially resolved with logstash-plugins/logstash-input-snmp#59
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.
It looks like it is fixed with #59, but we are leaving the ticket open pending verification from the person who reported it.
* Release notes for 7.2 (elastic#10863) * add release notes for 7.3.0 * Fix link formatting
* Release notes for 7.2 (elastic#10863) * add release notes for 7.3.0 * Fix link formatting
* Release notes for 7.2 (#10863) * 7.2.1 Release notes (#10959) * Add coming tag to 7.2.1 release notes (#10968) * Updated release notes for 7.2.1 (#10996) * Release notes 7.3.0 (#10993) * Release notes for 7.2 (#10863) * add release notes for 7.3.0 * Fix link formatting * Release notes draft for 7.3.1 (#11038) * Release notes draft for 7.3.2 (#11128) * Update release notes for 7.3.2 * Update releasenotes.asciidoc
* Release notes for 7.2 (#10863) * 7.2.1 Release notes (#10959) * Add coming tag to 7.2.1 release notes (#10968) * Updated release notes for 7.2.1 (#10996) * Release notes 7.3.0 (#10993) * Release notes for 7.2 (#10863) * add release notes for 7.3.0 * Fix link formatting * Release notes draft for 7.3.1 (#11038) * Release notes draft for 7.3.2 (#11128) * Update release notes for 7.3.2 * Update releasenotes.asciidoc
Includes a cherry-pick of the 7.2.0 release notes