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

Updated DEVELOPER_GUIDE.md and workflows to reference changes to the supported JDKs. #338

Merged
merged 13 commits into from
Mar 11, 2022

Conversation

AWSHurneyt
Copy link
Collaborator

@AWSHurneyt AWSHurneyt commented Mar 10, 2022

Issue #, if available:
#242

Description of changes:

  1. Updated DEVELOPER_GUIDE.md to reference changes to the supported JDKs based on this doc.
  2. Updated workflows to build using JDK 11.

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.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>
Signed-off-by: AWSHurneyt <hurneyt@amazon.com>
Signed-off-by: AWSHurneyt <hurneyt@amazon.com>
Signed-off-by: AWSHurneyt <hurneyt@amazon.com>
Signed-off-by: AWSHurneyt <hurneyt@amazon.com>
@AWSHurneyt AWSHurneyt marked this pull request as ready for review March 10, 2022 19:47
@codecov-commenter
Copy link

codecov-commenter commented Mar 10, 2022

Codecov Report

Merging #338 (8a3d494) into main (944d1fa) will decrease coverage by 0.24%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #338      +/-   ##
============================================
- Coverage     78.50%   78.26%   -0.25%     
- Complexity      214      264      +50     
============================================
  Files           173      176       +3     
  Lines          6975     7223     +248     
  Branches        916      960      +44     
============================================
+ Hits           5476     5653     +177     
- Misses         1010     1070      +60     
- Partials        489      500      +11     
Impacted Files Coverage Δ
...in/kotlin/org/opensearch/alerting/model/Monitor.kt 88.67% <0.00%> (-0.29%) ⬇️
...nsearch/alerting/core/model/ClusterMetricsInput.kt 95.30% <0.00%> (ø)
...erting/settings/SupportedClusterMetricsSettings.kt 19.51% <0.00%> (ø)
.../util/SupportedClusterMetricsSettingsExtensions.kt 35.41% <0.00%> (ø)
...n/kotlin/org/opensearch/alerting/AlertingPlugin.kt 94.73% <0.00%> (+0.03%) ⬆️
.../alerting/transport/TransportIndexMonitorAction.kt 59.52% <0.00%> (+0.47%) ⬆️
...ain/kotlin/org/opensearch/alerting/InputService.kt 88.57% <0.00%> (+0.69%) ⬆️

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 944d1fa...8a3d494. Read the comment docs.

@AWSHurneyt AWSHurneyt changed the title Updated DEVELOPER_GUIDE.md to reference changes to the supported JDKs. Updated DEVELOPER_GUIDE.md and workflows to reference changes to the supported JDKs. Mar 10, 2022
DEVELOPER_GUIDE.md Outdated Show resolved Hide resolved
Signed-off-by: AWSHurneyt <hurneyt@amazon.com>
DEVELOPER_GUIDE.md Outdated Show resolved Hide resolved
DEVELOPER_GUIDE.md Outdated Show resolved Hide resolved
Signed-off-by: AWSHurneyt <hurneyt@amazon.com>
Signed-off-by: AWSHurneyt <hurneyt@amazon.com>
@AWSHurneyt
Copy link
Collaborator Author

Successfully ran ./gradlew build locally as well.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>
@AWSHurneyt AWSHurneyt merged commit 9487c00 into opensearch-project:main Mar 11, 2022
AWSHurneyt added a commit to AWSHurneyt/OpenSearch-Alerting that referenced this pull request Mar 11, 2022
…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>
AWSHurneyt added a commit that referenced this pull request Mar 11, 2022
* Added support for JDK 8 and 14. (#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. (#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 (#339)

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

* Implemented support for ClusterMetrics monitors (#221) (#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. (#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. (#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 added a commit to AWSHurneyt/OpenSearch-Alerting that referenced this pull request Mar 11, 2022
…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 added a commit that referenced this pull request Mar 11, 2022
* Add 1.2.4.0 release notes (#282)

* Add 1.2.4.0 release notes

Signed-off-by: Ashish Agrawal <ashisagr@amazon.com>

* Commented out flaky test in favor of known issue to unblock build (#331) (#332) (#333)

* Commented out flaky test in favor of known issue #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>

* Cherry-picking commits for 1.3 release from main to 1.x (#345)

* Added support for JDK 8 and 14. (#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. (#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 (#339)

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

* Implemented support for ClusterMetrics monitors (#221) (#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. (#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. (#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>

Co-authored-by: Ashish Agrawal <ashisagr@amazon.com>
Co-authored-by: Saurabh Singh <getsaurabh02@gmail.com>
Co-authored-by: Saurabh Singh <sisurab@amazon.com>
Co-authored-by: Aditya Jindal <13850971+aditjind@users.noreply.github.com>
Co-authored-by: Annie Lee <leeyun@amazon.com>
AWSHurneyt added a commit to AWSHurneyt/OpenSearch-Alerting that referenced this pull request Mar 30, 2022
…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>
Signed-off-by: AWSHurneyt <hurneyt@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants