-
Notifications
You must be signed in to change notification settings - Fork 282
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
Add the consolidated release notes for 2.0.0 release #2137
Conversation
Signed-off-by: Zelin Hao <zelinhao@amazon.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.
A few edits suggested.
|
||
### OpenSearch Performance Analyzer | ||
* Adds setting to enable/disable Thread Contention Monitoring ([#171](https://github.com/opensearch-project/performance-analyzer/pull/171)) | ||
* Fixes calculation of average thread blocked time and average thread waited time ([#118](https://github.com/opensearch-project/performance-analyzer-rca/pull/118)) |
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.
Since this bullet point is a fix to an existing feature, I would recommend putting it in as an enhancement.
|
||
### OpenSearch SQL | ||
* Add Kmeans and AD command documentation ([#493](https://github.com/opensearch-project/sql/pull/493)) | ||
* AD and Kmeans grammar edits ([#500](https://github.com/opensearch-project/sql/pull/500)) |
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.
Would the edits in this line item make more sense to go under "ENHANCEMENT"?
### OpenSearch Alerting | ||
* Add automated migration for Destinations to Notification configs ([#379](https://github.com/opensearch-project/alerting/pull/379])) | ||
* Integrate with Notifications plugin for Alerting backend ([#401](https://github.com/opensearch-project/alerting/pull/401])) | ||
* Integrate Document Level Alerting changes ([#410](https://github.com/opensearch-project/alerting/pull/410])) |
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.
Document level alerting should be under the feature section since it is a net new feature in alerting.
* Add automated migration for Destinations to Notification configs ([#379](https://github.com/opensearch-project/alerting/pull/379])) | ||
* Integrate with Notifications plugin for Alerting backend ([#401](https://github.com/opensearch-project/alerting/pull/401])) | ||
* Integrate Document Level Alerting changes ([#410](https://github.com/opensearch-project/alerting/pull/410])) | ||
* Alias support for Document Level Monitors ([#416](https://github.com/opensearch-project/alerting/pull/416])) |
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.
Alias support looks like a net new feature for alerting. Should it go under "FEATURES"?
* Add CI check for demo script ([#1690](https://github.com/opensearch-project/security/pull/1690)) | ||
* Introduce BWC tests in security plugin ([#1685](https://github.com/opensearch-project/security/pull/1685)) | ||
* Correct the step name in CI ([#1683](https://github.com/opensearch-project/security/pull/1683)) | ||
* Add support for DLS Term Lookup Queries ([#1541](https://github.com/opensearch-project/security/pull/1541)) |
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.
This bullet point (Add support for DLS term lookup queries) should go under "FEATURES".
Signed-off-by: Zelin Hao <zelinhao@amazon.com>
@elfisher I made some modification on this PR based on your review. Please help review it again. Thanks. |
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
Signed-off-by: Zelin Hao <zelinhao@amazon.com>
|
||
## Release Details | ||
|
||
OpenSearch and OpenSearch Dashboards 2.0.0 include the following features, enhancements, bug fixes, infrastructure, documentation, maintenance, and refactoring updates |
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.
You can add a colon to this line.
OpenSearch and OpenSearch Dashboards 2.0.0 include the following features, enhancements, bug fixes, infrastructure, documentation, maintenance, and refactoring updates | |
OpenSearch and OpenSearch Dashboards 2.0.0 includes the following features, enhancements, bug fixes, infrastructure, documentation, maintenance, and refactoring updates: |
|
||
OpenSearch and OpenSearch Dashboards 2.0.0 include the following features, enhancements, bug fixes, infrastructure, documentation, maintenance, and refactoring updates | ||
|
||
OpenSearch [Release Notes](https://github.com/opensearch-project/OpenSearch/blob/2.0/release-notes/opensearch.release-notes-2.0.0.md) |
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 and line 21 be a bulleted list?
Signed-off-by: Zelin Hao <zelinhao@amazon.com>
Signed-off-by: Zelin Hao zelinhao@amazon.com
Description
Add the consolidated release notes for 2.0.0 release.
Issues Resolved
Part of #2086
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.