-
Notifications
You must be signed in to change notification settings - Fork 67
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 release notes for v2.11.1 #602
Add release notes for v2.11.1 #602
Conversation
Signed-off-by: Chenyang Ji <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #602 +/- ##
============================================
- Coverage 69.35% 69.13% -0.23%
+ Complexity 372 370 -2
============================================
Files 44 44
Lines 2689 2689
Branches 172 172
============================================
- Hits 1865 1859 -6
- Misses 721 726 +5
- Partials 103 104 +1 ☔ View full report in Codecov by Sentry. |
Compatible with OpenSearch 2.11.1 | ||
|
||
### Infrastructure | ||
* Set Autopublish to true in Jenkins publish for performance-analyzer-commons repo [#45](https://github.com/opensearch-project/performance-analyzer-commons/pull/45) |
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 does not concern PA in anyway right? I believe we can remove it? More related to PA-commons repo
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 don't have dedicated release notes for rca and commons repo, they are all included in the release notes in PA. You can check previous release notes for more information.
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 make the call as maintainers but PA-commons is an independent component with its own release cycle and versioning. Would recommend to add release notes for that component so when users go and check what was new in PA-commons 1.2.0 that was used in this release, it should be there. For example: common-utils new features wont be included in dependent component release notes, etc.
### Maintenance | ||
* Update build.gradle to use isSnapshot logic [#521](https://github.com/opensearch-project/performance-analyzer-rca/pull/521) | ||
* Add separate metric for cluster manager service events and metrics [#579](https://github.com/opensearch-project/performance-analyzer/pull/579) | ||
* Add separate metric in commons repo for cluster manager service events and metrics [#51](https://github.com/opensearch-project/performance-analyzer-commons/pull/51) |
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.
Same as above
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.
Same as above
Signed-off-by: Chenyang Ji <[email protected]> (cherry picked from commit ba69ebf)
Signed-off-by: Chenyang Ji <[email protected]> (cherry picked from commit ba69ebf)
Is your feature request related to a problem? Please provide an existing Issue # , or describe.
Add release notes for v2.11.1
Describe the solution you are proposing
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Check List
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.