Skip to content

Conversation

@bowenlan-amzn
Copy link
Member

Issue #, if available:

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.

peterzhuamazon and others added 3 commits July 28, 2021 13:32
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: bowenlan-amzn <bowenlan23@gmail.com>
@codecov-commenter
Copy link

codecov-commenter commented Jul 28, 2021

Codecov Report

Attention: Patch coverage is 68.29268% with 52 lines in your changes missing coverage. Please review.

Project coverage is 78.17%. Comparing base (091fe9f) to head (e68dc8f).
Report is 51 commits behind head on 1.x.

Files with missing lines Patch % Lines
...action/removepolicy/TransportRemovePolicyAction.kt 27.58% 20 Missing and 1 partial ⚠️
...ndexstatemanagement/IndexStateManagementHistory.kt 57.14% 11 Missing and 1 partial ⚠️
...agement/indexstatemanagement/ISMTemplateService.kt 77.14% 2 Missing and 6 partials ⚠️
...dexmanagement/indexstatemanagement/model/Policy.kt 52.94% 5 Missing and 3 partials ⚠️
...nt/indexstatemanagement/ManagedIndexCoordinator.kt 87.50% 0 Missing and 3 partials ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##                1.x     #106      +/-   ##
============================================
- Coverage     78.58%   78.17%   -0.42%     
+ Complexity     1895     1893       -2     
============================================
  Files           255      255              
  Lines         10378    10470      +92     
  Branches       1534     1548      +14     
============================================
+ Hits           8156     8185      +29     
- Misses         1374     1432      +58     
- Partials        848      853       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bowenlan-amzn bowenlan-amzn deleted the 1.x branch September 3, 2021 20:49
wuychn pushed a commit to ochprince/index-management that referenced this pull request Mar 16, 2023
…anagement (opensearch-project#106)

* opensearch-project#103 Attempt allocation implementation

* add allocation multinode tests

* add spacing to conditions

* add missing license

* verify node allocation during tests

Co-authored-by: Jakub Grzeska <jakub.grzeska@grapeup.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.

4 participants