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

Backport to 1.x : Commented out flaky test in favor of known issue to unblock build (#331) #332

Merged
merged 1 commit into from
Mar 10, 2022

Conversation

getsaurabh02
Copy link
Member

Issue #, if available:
Backport #331 to 1.x

Description of changes:

CheckList:
[ ] 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.

…ensearch-project#331)

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

Signed-off-by: Saurabh Singh <[email protected]>

* Commenting out another test

Signed-off-by: Aditya Jindal <[email protected]>

Co-authored-by: Saurabh Singh <[email protected]>
Co-authored-by: Aditya Jindal <[email protected]>
@getsaurabh02 getsaurabh02 requested a review from a team March 10, 2022 07:26
@codecov-commenter
Copy link

Codecov Report

Merging #332 (cab9d1d) into 1.x (31c2499) will decrease coverage by 0.07%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##                1.x     #332      +/-   ##
============================================
- Coverage     78.55%   78.48%   -0.08%     
+ Complexity      217      215       -2     
============================================
  Files           173      173              
  Lines          6975     6975              
  Branches        916      916              
============================================
- Hits           5479     5474       -5     
- Misses         1007     1010       +3     
- Partials        489      491       +2     
Impacted Files Coverage Δ
...ation/factory/CustomWebhookDestinationFactory.java 75.00% <0.00%> (-25.00%) ⬇️
...ting/destination/message/CustomWebhookMessage.java 80.00% <0.00%> (-1.54%) ⬇️
...ting/destination/client/DestinationHttpClient.java 86.36% <0.00%> (-1.52%) ⬇️
.../kotlin/org/opensearch/alerting/core/JobSweeper.kt 71.50% <0.00%> (-0.50%) ⬇️
...ain/kotlin/org/opensearch/alerting/AlertService.kt 78.53% <0.00%> (-0.49%) ⬇️
...ing/destination/client/DestinationEmailClient.java 77.50% <0.00%> (+5.00%) ⬆️

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 31c2499...cab9d1d. Read the comment docs.

@adityaj1107 adityaj1107 merged commit cfdbe73 into opensearch-project:1.x Mar 10, 2022
getsaurabh02 added a commit to getsaurabh02/alerting that referenced this pull request Mar 10, 2022
…ensearch-project#331) (opensearch-project#332)

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

Signed-off-by: Saurabh Singh <[email protected]>

* Commenting out another test

Signed-off-by: Aditya Jindal <[email protected]>

Co-authored-by: Saurabh Singh <[email protected]>
Co-authored-by: Aditya Jindal <[email protected]>

Co-authored-by: Saurabh Singh <[email protected]>
Co-authored-by: Aditya Jindal <[email protected]>
getsaurabh02 added a commit that referenced this pull request Mar 10, 2022
…) (#332) (#333)

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

Signed-off-by: Saurabh Singh <[email protected]>

* Commenting out another test

Signed-off-by: Aditya Jindal <[email protected]>

Co-authored-by: Saurabh Singh <[email protected]>
Co-authored-by: Aditya Jindal <[email protected]>

Co-authored-by: Saurabh Singh <[email protected]>
Co-authored-by: Aditya Jindal <[email protected]>

Co-authored-by: Saurabh Singh <[email protected]>
Co-authored-by: Aditya Jindal <[email protected]>
AWSHurneyt pushed a commit to AWSHurneyt/OpenSearch-Alerting that referenced this pull request Mar 11, 2022
…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 <[email protected]>

* Commenting out another test

Signed-off-by: Aditya Jindal <[email protected]>

Co-authored-by: Saurabh Singh <[email protected]>
Co-authored-by: Aditya Jindal <[email protected]>

Co-authored-by: Saurabh Singh <[email protected]>
Co-authored-by: Aditya Jindal <[email protected]>

Co-authored-by: Saurabh Singh <[email protected]>
Co-authored-by: Aditya Jindal <[email protected]>
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 <[email protected]>

* 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 <[email protected]>

* Commenting out another test

Signed-off-by: Aditya Jindal <[email protected]>

Co-authored-by: Saurabh Singh <[email protected]>
Co-authored-by: Aditya Jindal <[email protected]>

Co-authored-by: Saurabh Singh <[email protected]>
Co-authored-by: Aditya Jindal <[email protected]>

Co-authored-by: Saurabh Singh <[email protected]>
Co-authored-by: Aditya Jindal <[email protected]>

* 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 <[email protected]>

* Added support for JDK 8 and 14.

Signed-off-by: AWSHurneyt <[email protected]>

* Added 1.3 release notes. (#336)

* Added 1.3 release notes.

Signed-off-by: AWSHurneyt <[email protected]>

* Added 1.3 release notes.

Signed-off-by: AWSHurneyt <[email protected]>

* Add backport and auto delete workflow (#339)

Signed-off-by: Annie Lee <[email protected]>

* Implemented support for ClusterMetrics monitors (#221) (#342)

* Implemented support for LocalUriInput monitors.

Signed-off-by: AWSHurneyt <[email protected]>

* 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 <[email protected]>

* Fixed ktlint errors.

Signed-off-by: AWSHurneyt <[email protected]>

* Implemented randomClusterMetricsMonitor test helper, and refactored tests accordingly.

Signed-off-by: AWSHurneyt <[email protected]>

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

Signed-off-by: AWSHurneyt <[email protected]>

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

Signed-off-by: AWSHurneyt <[email protected]>

* Refactored supported JSON payload to return all response fields.

Signed-off-by: AWSHurneyt <[email protected]>

* Refactored nodes stats request object to return all metrics.

Signed-off-by: AWSHurneyt <[email protected]>

* Refactored nodes stats request object to return all metrics.

Signed-off-by: AWSHurneyt <[email protected]>

* Removing unused connectionTimeout and socketTimeout params.

Signed-off-by: AWSHurneyt <[email protected]>

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

* Added 1.3 release notes.

Signed-off-by: AWSHurneyt <[email protected]>

* Added 1.3 release notes.

Signed-off-by: AWSHurneyt <[email protected]>

* Added cluster metrics monitor to release notes.

Signed-off-by: AWSHurneyt <[email protected]>

* 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 <[email protected]>

* Added support for JDK 8 and 14.

Signed-off-by: AWSHurneyt <[email protected]>

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

Signed-off-by: AWSHurneyt <[email protected]>

* Added additional release note.

Signed-off-by: AWSHurneyt <[email protected]>

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

Signed-off-by: AWSHurneyt <[email protected]>

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

Signed-off-by: AWSHurneyt <[email protected]>

* Updated workflows to build using JDK 11.

Signed-off-by: AWSHurneyt <[email protected]>

* Removing redundant JDK setup step.

Signed-off-by: AWSHurneyt <[email protected]>

* Fixed guide wording.

Signed-off-by: AWSHurneyt <[email protected]>

* Fixed guide wording.

Signed-off-by: AWSHurneyt <[email protected]>

* Reverted bwc workflow JDK.

Signed-off-by: AWSHurneyt <[email protected]>

Co-authored-by: Annie Lee <[email protected]>

Co-authored-by: Ashish Agrawal <[email protected]>
Co-authored-by: Saurabh Singh <[email protected]>
Co-authored-by: Saurabh Singh <[email protected]>
Co-authored-by: Aditya Jindal <[email protected]>
Co-authored-by: Annie Lee <[email protected]>
AWSHurneyt pushed a commit to AWSHurneyt/OpenSearch-Alerting that referenced this pull request Mar 30, 2022
…ensearch-project#331) (opensearch-project#332)

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

Signed-off-by: Saurabh Singh <[email protected]>

* Commenting out another test

Signed-off-by: Aditya Jindal <[email protected]>

Co-authored-by: Saurabh Singh <[email protected]>
Co-authored-by: Aditya Jindal <[email protected]>

Co-authored-by: Saurabh Singh <[email protected]>
Co-authored-by: Aditya Jindal <[email protected]>
Signed-off-by: AWSHurneyt <[email protected]>
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.

4 participants