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

Upgrade to opensearch 2.0.0 alpha1 #1741

Merged

Conversation

cliu123
Copy link
Member

@cliu123 cliu123 commented Apr 4, 2022

Description

  • Category (Enhancement, New feature, Bug fix, Test fix, Refactoring, Maintenance, Documentation) Maintenance

Issues Resolved

#1642

Is this a backport? If so, please add backport PR # and/or commits #

Testing

UTs

Check List

  • 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.

@cliu123 cliu123 requested a review from a team April 4, 2022 21:25
@cliu123 cliu123 force-pushed the upgrade_to_opensearch_2.0.0_alpha1 branch 7 times, most recently from d984387 to d71db1b Compare April 5, 2022 15:27
@peternied
Copy link
Member

@cliu123 I've just merged the test changes from this PR into #1742

@peternied
Copy link
Member

@peterzhuamazon I believe we have a functional version of the build qualifiers #1742 - what do you think of that change?

@cliu123
Copy link
Member Author

cliu123 commented Apr 5, 2022

@cliu123 I've just merged the test changes from this PR into #1742

@peternied Can we avoid mixing up changes? You don't have to merge the changes in the PR to PR#1742 when this PR is in WIP.

peterzhuamazon and others added 5 commits April 6, 2022 10:04
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: cliu123 <lc12251109@gmail.com>
Signed-off-by: cliu123 <lc12251109@gmail.com>
Signed-off-by: cliu123 <lc12251109@gmail.com>
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
@cliu123 cliu123 force-pushed the upgrade_to_opensearch_2.0.0_alpha1 branch from dbbb490 to 640a62a Compare April 6, 2022 17:04
Signed-off-by: cliu123 <lc12251109@gmail.com>
@codecov-commenter
Copy link

codecov-commenter commented Apr 6, 2022

Codecov Report

Merging #1741 (ec4ced3) into main (2972bd1) will increase coverage by 0.01%.
The diff coverage is 0.00%.

❗ Current head ec4ced3 differs from pull request most recent head f6246a8. Consider uploading reports for the commit f6246a8 to get more accurate results

@@             Coverage Diff              @@
##               main    #1741      +/-   ##
============================================
+ Coverage     60.38%   60.40%   +0.01%     
- Complexity     3195     3197       +2     
============================================
  Files           253      253              
  Lines         18095    18093       -2     
  Branches       3245     3245              
============================================
+ Hits          10927    10929       +2     
+ Misses         5586     5583       -3     
+ Partials       1582     1581       -1     
Impacted Files Coverage Δ
...iance/ComplianceIndexingOperationListenerImpl.java 60.86% <0.00%> (ø)
...nsearch/security/compliance/FieldReadCallback.java 55.05% <0.00%> (ø)
...security/configuration/DlsFlsFilterLeafReader.java 58.62% <ø> (+0.74%) ⬆️

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 2972bd1...f6246a8. Read the comment docs.

@cliu123
Copy link
Member Author

cliu123 commented Apr 6, 2022

The PR is being blocked by opensearch-project/OpenSearch#2769

Signed-off-by: cliu123 <lc12251109@gmail.com>
@cliu123 cliu123 force-pushed the upgrade_to_opensearch_2.0.0_alpha1 branch 2 times, most recently from 9fcb6e1 to 7c88c43 Compare April 6, 2022 20:03
Signed-off-by: cliu123 <lc12251109@gmail.com>
@cliu123 cliu123 force-pushed the upgrade_to_opensearch_2.0.0_alpha1 branch 3 times, most recently from ec4ced3 to f6246a8 Compare April 6, 2022 21:15
@peternied peternied merged commit 2744081 into opensearch-project:main Apr 7, 2022
@peternied peternied mentioned this pull request Apr 7, 2022
@cliu123 cliu123 deleted the upgrade_to_opensearch_2.0.0_alpha1 branch April 19, 2022 21:59
@cliu123 cliu123 mentioned this pull request Apr 20, 2022
3 tasks
peternied added a commit that referenced this pull request Sep 23, 2022
Incorperating changes from
#1741

Specifically build.gradle changes, see 2744081#diff-49a96e7eea8a94af862798a45174e6ac43eb4f8b4bd40759b5da63ba31ec3ef7

Signed-off-by: Peter Nied <petern@amazon.com>
peternied added a commit that referenced this pull request Sep 23, 2022
Incorperating changes from
#1741

Specifically build.gradle changes, see 2744081#diff-49a96e7eea8a94af862798a45174e6ac43eb4f8b4bd40759b5da63ba31ec3ef7

Signed-off-by: Peter Nied <petern@amazon.com>
peternied added a commit that referenced this pull request Sep 24, 2022
Incorperating changes from
#1741

Specifically build.gradle changes, see 2744081#diff-49a96e7eea8a94af862798a45174e6ac43eb4f8b4bd40759b5da63ba31ec3ef7

Signed-off-by: Peter Nied <petern@amazon.com>
peternied added a commit that referenced this pull request Sep 28, 2022
* Staging for version increment automation (#1932)

Incorporating changes from
#1741

Specifically build.gradle changes, see 2744081#diff-49a96e7eea8a94af862798a45174e6ac43eb4f8b4bd40759b5da63ba31ec3ef7

Signed-off-by: pgodithi <pgodithi@amazon.com>
Signed-off-by: Peter Nied <petern@amazon.com>
Co-authored-by: Prudhvi Godithi <pgodithi@amazon.com>
(cherry picked from commit 235d256)
wuychn pushed a commit to ochprince/security that referenced this pull request Mar 16, 2023
* Security plugin qualifier support default to alpha1

Signed-off-by: cliu123 <lc12251109@gmail.com>
Co-authored-by: Peter Zhu <zhujiaxi@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.

5 participants