Skip to content
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

Merged
merged 4 commits into from
May 26, 2022

Conversation

zelinh
Copy link
Member

@zelinh zelinh commented May 20, 2022

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.

Signed-off-by: Zelin Hao <zelinhao@amazon.com>
@zelinh zelinh requested a review from a team as a code owner May 20, 2022 21:35
@zelinh zelinh mentioned this pull request May 20, 2022
46 tasks
Copy link
Contributor

@elfisher elfisher left a 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))
Copy link
Contributor

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))
Copy link
Contributor

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]))
Copy link
Contributor

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]))
Copy link
Contributor

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))
Copy link
Contributor

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>
@codecov-commenter
Copy link

codecov-commenter commented May 23, 2022

Codecov Report

Merging #2137 (92c734c) into main (438e09d) will increase coverage by 5.43%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #2137      +/-   ##
=============================================
+ Coverage     94.56%   100.00%   +5.43%     
=============================================
  Files           205         6     -199     
  Lines          4254       105    -4149     
  Branches         29        19      -10     
=============================================
- Hits           4023       105    -3918     
+ Misses          225         0     -225     
+ Partials          6         0       -6     
Impacted Files Coverage Δ
src/test_workflow/perf_test/perf_test_cluster.py
src/test_workflow/integ_test/distribution.py
tests/jenkins/jobs/RpmMetaValidation_Jenkinsfile
tests/jenkins/jobs/BuildFailureMessage_Jenkinsfile
src/jenkins/InputManifest.groovy
src/paths/script_finder.py
...workflow/integ_test/integ_test_start_properties.py
src/test_workflow/test_recorder/log_recorder.py
src/manifests/component_manifest.py
src/test_workflow/test_recorder/test_recorder.py
... and 189 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 438e09d...92c734c. Read the comment docs.

@zelinh
Copy link
Member Author

zelinh commented May 23, 2022

@elfisher I made some modification on this PR based on your review. Please help review it again. Thanks.

@zelinh zelinh requested a review from elfisher May 23, 2022 20:21
Copy link
Contributor

@elfisher elfisher left a 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
Copy link

@Naarcha-AWS Naarcha-AWS May 26, 2022

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.

Suggested change
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)

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>
@bbarani bbarani merged commit 44cac5a into opensearch-project:main May 26, 2022
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