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

Updates alerting version to 1.2 #192

Merged
merged 2 commits into from
Oct 4, 2021

Conversation

downsrob
Copy link
Contributor

@downsrob downsrob commented Oct 1, 2021

Signed-off-by: Robert Downs [email protected]

Issue #, if available:
opensearch-project/opensearch-build#663
Description of changes:
Updates alerting version to 1.2. Changes the gradle build file, as well as the github test and build workflows.
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.

@codecov-commenter
Copy link

Codecov Report

Merging #192 (b2c41ce) into main (a91df8f) will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #192      +/-   ##
============================================
+ Coverage     78.82%   78.85%   +0.02%     
- Complexity      214      215       +1     
============================================
  Files           172      172              
  Lines          6948     6948              
  Branches        903      903              
============================================
+ Hits           5477     5479       +2     
+ Misses          987      986       -1     
+ Partials        484      483       -1     
Impacted Files Coverage Δ
.../kotlin/org/opensearch/alerting/core/JobSweeper.kt 71.72% <0.00%> (-0.53%) ⬇️
...lerting/transport/TransportExecuteMonitorAction.kt 73.33% <0.00%> (+2.22%) ⬆️
...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 a91df8f...b2c41ce. Read the comment docs.

@downsrob downsrob merged commit 63e2a3e into opensearch-project:main Oct 4, 2021
@downsrob downsrob deleted the update-to-1.2 branch October 4, 2021 15:44
rishabhmaurya pushed a commit to rishabhmaurya/alerting-1 that referenced this pull request Nov 8, 2021
* Updates alerting version to 1.2

* Adds snapshot repo to the repository file

Signed-off-by: Clay Downs <[email protected]>
rishabhmaurya added a commit that referenced this pull request Nov 9, 2021
* Updates alerting version to 1.2 (#192)

* Updates alerting version to 1.2

* Adds snapshot repo to the repository file

Signed-off-by: Clay Downs <[email protected]>

* Update build to use public Maven repo (#184)

Signed-off-by: Abbas Hussain <[email protected]>

* Publish notification JARs checksums. (#196)

* Publish notification JARs checksums.

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

* Remove sonatype staging.

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

* Updates testCompile mockito version to match OpenSearch changes (#204)

Signed-off-by: Clay Downs <[email protected]>

* Update maven publication to include cksums. (#224)

This change adds a task to publish to a local staging repo under build/ that includes cksums.  It also updates build.sh to use this new task and copy the contents of the staging repo to the output directory.
The maven publish plugin will not include these cksums when publishing to maven local but will when published to a separate folder.

Signed-off-by: Marc Handalian <[email protected]>

* Add release notes for 1.2.0.0 release (#225)

* Create opensearch-alerting.release-notes-1.2.0.0.md

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

* Update opensearch-alerting.release-notes-1.2.0.0.md

* Update opensearch-alerting.release-notes-1.2.0.0.md

* Add backwards compatibility tests (#199)

* Initial commit for BWC tests

Signed-off-by: Mohammad Qureshi <[email protected]>

* Update bwc test to check Monitor stats and add bwc tests to GitHub Actions

Signed-off-by: Mohammad Qureshi <[email protected]>

* Use current version plugin bundle from build for bwc tests instead of manually uploading

Signed-off-by: Mohammad Qureshi <[email protected]>

* Update mockito-core dependency to 3.12.4 to prevent conflict

Signed-off-by: Mohammad Qureshi <[email protected]>

* Remove disabling security manager flag when running BWC tests

Signed-off-by: Mohammad Qureshi <[email protected]>

Co-authored-by: Clay Downs <[email protected]>
Co-authored-by: Abbas Hussain <[email protected]>
Co-authored-by: Daniel Doubrovkine (dB.) <[email protected]>
Co-authored-by: Marc Handalian <[email protected]>
Co-authored-by: Annie Lee <[email protected]>
Co-authored-by: Mohammad Qureshi <[email protected]>
rishabhmaurya added a commit that referenced this pull request Nov 9, 2021
* Cherry-pick commits to 1.x (#227)

* Update copyright notice (#222)

Signed-off-by: Mohammad Qureshi <[email protected]>

* Admin Users must be able to access all monitors #139 (#220)

* Admin Users must be able to access all monitors #139

* Refactored

Co-authored-by: Mohammad Qureshi <[email protected]>
Co-authored-by: Sriram <[email protected]>

* Updates alerting version to 1.2 (#192)

* Updates alerting version to 1.2

* Adds snapshot repo to the repository file

Signed-off-by: Clay Downs <[email protected]>

* Update build to use public Maven repo (#184)

Signed-off-by: Abbas Hussain <[email protected]>

* Publish notification JARs checksums. (#196)

* Publish notification JARs checksums.

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

* Remove sonatype staging.

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

* Updates testCompile mockito version to match OpenSearch changes (#204)

Signed-off-by: Clay Downs <[email protected]>

* Update maven publication to include cksums. (#224)

This change adds a task to publish to a local staging repo under build/ that includes cksums.  It also updates build.sh to use this new task and copy the contents of the staging repo to the output directory.
The maven publish plugin will not include these cksums when publishing to maven local but will when published to a separate folder.

Signed-off-by: Marc Handalian <[email protected]>

* Add release notes for 1.2.0.0 release (#225)

* Create opensearch-alerting.release-notes-1.2.0.0.md

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

* Update opensearch-alerting.release-notes-1.2.0.0.md

* Update opensearch-alerting.release-notes-1.2.0.0.md

* Add backwards compatibility tests (#199)

* Initial commit for BWC tests

Signed-off-by: Mohammad Qureshi <[email protected]>

* Update bwc test to check Monitor stats and add bwc tests to GitHub Actions

Signed-off-by: Mohammad Qureshi <[email protected]>

* Use current version plugin bundle from build for bwc tests instead of manually uploading

Signed-off-by: Mohammad Qureshi <[email protected]>

* Update mockito-core dependency to 3.12.4 to prevent conflict

Signed-off-by: Mohammad Qureshi <[email protected]>

* Remove disabling security manager flag when running BWC tests

Signed-off-by: Mohammad Qureshi <[email protected]>

Co-authored-by: Mohammad Qureshi <[email protected]>
Co-authored-by: Sriram <[email protected]>
Co-authored-by: Clay Downs <[email protected]>
Co-authored-by: Abbas Hussain <[email protected]>
Co-authored-by: Daniel Doubrovkine (dB.) <[email protected]>
Co-authored-by: Marc Handalian <[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
* Updates alerting version to 1.2 (opensearch-project#192)

* Updates alerting version to 1.2

* Adds snapshot repo to the repository file

Signed-off-by: Clay Downs <[email protected]>

* Update build to use public Maven repo (opensearch-project#184)

Signed-off-by: Abbas Hussain <[email protected]>

* Publish notification JARs checksums. (opensearch-project#196)

* Publish notification JARs checksums.

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

* Remove sonatype staging.

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

* Updates testCompile mockito version to match OpenSearch changes (opensearch-project#204)

Signed-off-by: Clay Downs <[email protected]>

* Update maven publication to include cksums. (opensearch-project#224)

This change adds a task to publish to a local staging repo under build/ that includes cksums.  It also updates build.sh to use this new task and copy the contents of the staging repo to the output directory.
The maven publish plugin will not include these cksums when publishing to maven local but will when published to a separate folder.

Signed-off-by: Marc Handalian <[email protected]>

* Add release notes for 1.2.0.0 release (opensearch-project#225)

* Create opensearch-alerting.release-notes-1.2.0.0.md

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

* Update opensearch-alerting.release-notes-1.2.0.0.md

* Update opensearch-alerting.release-notes-1.2.0.0.md

* Add backwards compatibility tests (opensearch-project#199)

* Initial commit for BWC tests

Signed-off-by: Mohammad Qureshi <[email protected]>

* Update bwc test to check Monitor stats and add bwc tests to GitHub Actions

Signed-off-by: Mohammad Qureshi <[email protected]>

* Use current version plugin bundle from build for bwc tests instead of manually uploading

Signed-off-by: Mohammad Qureshi <[email protected]>

* Update mockito-core dependency to 3.12.4 to prevent conflict

Signed-off-by: Mohammad Qureshi <[email protected]>

* Remove disabling security manager flag when running BWC tests

Signed-off-by: Mohammad Qureshi <[email protected]>

Co-authored-by: Clay Downs <[email protected]>
Co-authored-by: Abbas Hussain <[email protected]>
Co-authored-by: Daniel Doubrovkine (dB.) <[email protected]>
Co-authored-by: Marc Handalian <[email protected]>
Co-authored-by: Annie Lee <[email protected]>
Co-authored-by: Mohammad Qureshi <[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.

3 participants