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

version upgrade to 2.1.0 #389

Merged

Conversation

rupanshisharma
Copy link
Contributor

@rupanshisharma rupanshisharma commented Jun 23, 2022

Signed-off-by: Rupanshi Sharma [email protected]

Issue #: 385

Version Upgrade to 2.1.0

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.

Signed-off-by: Rupanshi Sharma <[email protected]>
@rupanshisharma rupanshisharma requested review from bowenlan-amzn, downsrob and a team and removed request for downsrob June 23, 2022 00:16
Rupanshi Sharma added 2 commits June 23, 2022 08:56
…pe in a test class to fix the unresolved method name error

Signed-off-by: Rupanshi Sharma <[email protected]>
@rupanshisharma rupanshisharma requested a review from downsrob June 23, 2022 22:16
Signed-off-by: Rupanshi Sharma <[email protected]>
downsrob
downsrob previously approved these changes Jun 23, 2022
@bowenlan-amzn
Copy link
Member

Notification jar seems to be ready, re-trigger all failed workflows

@codecov-commenter
Copy link

codecov-commenter commented Jun 24, 2022

Codecov Report

Merging #389 (54f4f08) into main (c2646f1) will decrease coverage by 0.09%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #389      +/-   ##
============================================
- Coverage     75.40%   75.30%   -0.10%     
- Complexity     2159     2161       +2     
============================================
  Files           263      263              
  Lines         12429    12429              
  Branches       1975     1975              
============================================
- Hits           9372     9360      -12     
- Misses         1975     1984       +9     
- Partials       1082     1085       +3     
Impacted Files Coverage Δ
...statemanagement/model/destination/CustomWebhook.kt 67.14% <0.00%> (-28.58%) ⬇️
...gement/indexstatemanagement/action/ShrinkAction.kt 69.11% <0.00%> (-2.95%) ⬇️
.../rollup/action/start/TransportStartRollupAction.kt 72.94% <0.00%> (-2.36%) ⬇️
...management/indexstatemanagement/MetadataService.kt 62.68% <0.00%> (-2.24%) ⬇️
.../opensearch/indexmanagement/rollup/model/Rollup.kt 86.04% <0.00%> (ø)
...nt/indexstatemanagement/ManagedIndexCoordinator.kt 69.29% <0.00%> (ø)
...earch/indexmanagement/transform/model/Transform.kt 85.89% <0.00%> (+0.42%) ⬆️
...ent/indexstatemanagement/util/ManagedIndexUtils.kt 77.87% <0.00%> (+1.76%) ⬆️
...anagement/indexstatemanagement/model/Transition.kt 65.75% <0.00%> (+2.73%) ⬆️
...arch/indexmanagement/rollup/RollupSearchService.kt 61.11% <0.00%> (+3.70%) ⬆️
... and 2 more

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 c2646f1...54f4f08. Read the comment docs.

@rupanshisharma
Copy link
Contributor Author

blocking PR: #389

@rupanshisharma rupanshisharma requested a review from downsrob June 24, 2022 19:36
@rupanshisharma rupanshisharma merged commit 824105d into opensearch-project:main Jun 24, 2022
@rupanshisharma rupanshisharma mentioned this pull request Jun 24, 2022
22 tasks
wuychn pushed a commit to ochprince/index-management that referenced this pull request Mar 16, 2023
* version upgrade to 2.1.0

Signed-off-by: Rupanshi Sharma <[email protected]>

* Renames master/Master to clusterManager/ClusterManager as mentioned here: opensearch-project/OpenSearch#3542

Signed-off-by: Rupanshi Sharma <[email protected]>

* Change XContentType.fromMediaTypeOrFormat to XContentType.fromMediaType in a test class to fix the unresolved method name error

Signed-off-by: Rupanshi Sharma <[email protected]>

* Upgrade gradle jar

Signed-off-by: Rupanshi Sharma <[email protected]>

* Revert "Renames master/Master to clusterManager/ClusterManager as mentioned here: opensearch-project/OpenSearch#3542"

This reverts commit 01dd024.

Signed-off-by: Rupanshi Sharma <[email protected]>

Co-authored-by: Rupanshi Sharma <[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.

5 participants