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

Run a GHA against OpenSearch 2.0.0 #1467

Merged

Conversation

dlvenable
Copy link
Member

@dlvenable dlvenable commented Jun 2, 2022

Description

Now that OpenSearch 2.0.0 is released, run a GHA to verify that Data Prepper runs against OpenSearch 2.0.0. Also bumped 1.3.0 to 1.3.1 which is the current latest in that series.

Issues Resolved

N/A

Check List

  • New functionality includes testing.
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed with a real name per the DCO

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.

… 1.3.1 instead of 1.3.0.

Signed-off-by: David Venable <dlv@amazon.com>
@dlvenable dlvenable requested a review from a team as a code owner June 2, 2022 17:09
@codecov-commenter
Copy link

codecov-commenter commented Jun 2, 2022

Codecov Report

Merging #1467 (fd0a018) into main (7bc7dbf) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #1467   +/-   ##
=========================================
  Coverage     94.25%   94.25%           
  Complexity     1179     1179           
=========================================
  Files           165      165           
  Lines          3377     3377           
  Branches        276      276           
=========================================
  Hits           3183     3183           
  Misses          138      138           
  Partials         56       56           

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 bd389dd...fd0a018. Read the comment docs.

asifsmohammed
asifsmohammed previously approved these changes Jun 3, 2022
sshivanii
sshivanii previously approved these changes Jun 3, 2022
@@ -10,7 +10,7 @@ jobs:
strategy:
matrix:
java: [11]
opensearch: [1.0.1, 1.1.0, 1.2.4, 1.3.0]
opensearch: [1.0.1, 1.1.0, 1.2.4, 1.3.1, 2.0.0]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For what it's worth 1.3.2 is the most recent in the 1.3.x line - https://opensearch.org/lines/1x.html

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for noting that! I have pushed a new commit to use 1.3.2 instead.

Signed-off-by: David Venable <dlv@amazon.com>
@dlvenable dlvenable dismissed stale reviews from sshivanii and asifsmohammed via fd0a018 June 3, 2022 18:24
@dlvenable dlvenable merged commit be35437 into opensearch-project:main Jun 3, 2022
@dlvenable dlvenable deleted the opensearch-2.0.0-integration-test branch June 9, 2022 00:50
finnroblin pushed a commit to finnroblin/data-prepper that referenced this pull request Jul 11, 2022
Run a GHA against OpenSearch 2.0.0 to verify it works. Updated to use 1.3.2 instead of 1.3.0.

Signed-off-by: David Venable <dlv@amazon.com>
Signed-off-by: Finn Roblin <finnrobl@amazon.com>
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.

None yet

5 participants