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

1.3 release - porting 1.x to 1.3 #347

Merged
merged 4 commits into from
Mar 11, 2022

Conversation

AWSHurneyt
Copy link
Collaborator

Issue #, if available:
#314

Description of changes:
Porting 1.x to 1.3.

CheckList:
[x] Commits are signed per the DCO using --signoff

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.

lezzago and others added 3 commits March 10, 2022 17:28
* Add 1.2.4.0 release notes

Signed-off-by: Ashish Agrawal <ashisagr@amazon.com>
…ensearch-project#331) (opensearch-project#332) (opensearch-project#333)

* Commented out flaky test in favor of known issue opensearch-project#300 to unblock build process.

Signed-off-by: Saurabh Singh <sisurab@amazon.com>

* Commenting out another test

Signed-off-by: Aditya Jindal <aditjind@amazon.com>

Co-authored-by: Saurabh Singh <sisurab@amazon.com>
Co-authored-by: Aditya Jindal <aditjind@amazon.com>

Co-authored-by: Saurabh Singh <sisurab@amazon.com>
Co-authored-by: Aditya Jindal <13850971+aditjind@users.noreply.github.com>

Co-authored-by: Saurabh Singh <sisurab@amazon.com>
Co-authored-by: Aditya Jindal <13850971+aditjind@users.noreply.github.com>
…roject#345)

* Added support for JDK 8 and 14. (opensearch-project#335)

* Added support for JDK 8 and 14.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Added support for JDK 8 and 14.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Added 1.3 release notes. (opensearch-project#336)

* Added 1.3 release notes.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Added 1.3 release notes.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Add backport and auto delete workflow (opensearch-project#339)

Signed-off-by: Annie Lee <leeyun@amazon.com>

* Implemented support for ClusterMetrics monitors (opensearch-project#221) (opensearch-project#342)

* Implemented support for LocalUriInput monitors.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Refactored feature naming convention from LocalUriInput to ClusterMetricsInput. Added Cluster Metrics as a new monitor type to align with frontend experience.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Fixed ktlint errors.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Implemented randomClusterMetricsMonitor test helper, and refactored tests accordingly.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Renamed some assets to align with the new name for this feature.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Refactored cluster metrics feature to remove support for Cat repositories API.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Refactored supported JSON payload to return all response fields.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Refactored nodes stats request object to return all metrics.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Refactored nodes stats request object to return all metrics.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Removing unused connectionTimeout and socketTimeout params.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Added cluster metrics monitor to release notes. (opensearch-project#343)

* Added 1.3 release notes.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Added 1.3 release notes.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Added cluster metrics monitor to release notes.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Updated DEVELOPER_GUIDE.md and workflows to reference changes to the supported JDKs. (opensearch-project#338)

* Added support for JDK 8 and 14.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Added support for JDK 8 and 14.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Updated DEVELOPER_GUIDE.md to reference changes to the supported JDKs.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Added additional release note.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Updated DEVELOPER_GUIDE.md to reference changes to the supported JDKs.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Updated DEVELOPER_GUIDE.md to reference changes to the supported JDKs.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Updated workflows to build using JDK 11.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Removing redundant JDK setup step.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Fixed guide wording.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Fixed guide wording.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Reverted bwc workflow JDK.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

Co-authored-by: Annie Lee <leeyun@amazon.com>
@AWSHurneyt AWSHurneyt marked this pull request as ready for review March 11, 2022 01:51
@AWSHurneyt AWSHurneyt requested a review from a team March 11, 2022 01:51
@codecov-commenter
Copy link

codecov-commenter commented Mar 11, 2022

Codecov Report

Merging #347 (5ef3279) into 1.3 (515b8fd) will decrease coverage by 0.20%.
The diff coverage is 70.91%.

Impacted file tree graph

@@             Coverage Diff              @@
##                1.3     #347      +/-   ##
============================================
- Coverage     78.43%   78.23%   -0.21%     
- Complexity      214      264      +50     
============================================
  Files           173      176       +3     
  Lines          6975     7223     +248     
  Branches        916      960      +44     
============================================
+ Hits           5471     5651     +180     
- Misses         1013     1070      +57     
- Partials        491      502      +11     
Impacted Files Coverage Δ
...erting/settings/SupportedClusterMetricsSettings.kt 19.51% <19.51%> (ø)
.../util/SupportedClusterMetricsSettingsExtensions.kt 35.41% <35.41%> (ø)
...ain/kotlin/org/opensearch/alerting/InputService.kt 88.57% <75.00%> (+0.69%) ⬆️
...in/kotlin/org/opensearch/alerting/model/Monitor.kt 88.67% <85.71%> (-0.29%) ⬇️
...nsearch/alerting/core/model/ClusterMetricsInput.kt 95.30% <95.30%> (ø)
...n/kotlin/org/opensearch/alerting/AlertingPlugin.kt 94.73% <100.00%> (+0.03%) ⬆️
.../alerting/transport/TransportIndexMonitorAction.kt 59.52% <0.00%> (+0.47%) ⬆️
...ain/kotlin/org/opensearch/alerting/AlertService.kt 78.53% <0.00%> (+0.48%) ⬆️
.../kotlin/org/opensearch/alerting/core/JobSweeper.kt 72.00% <0.00%> (+0.50%) ⬆️
... and 1 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 515b8fd...5ef3279. Read the comment docs.

@AWSHurneyt AWSHurneyt merged commit 9aa3d2c into opensearch-project:1.3 Mar 11, 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.

6 participants