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

[Backport 2.x] Fix bug with delete shard routing weights on node restart #8075

Merged
merged 1 commit into from
Jun 15, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 87e34a3 from #8057.

Signed-off-by: Rishab Nahata <[email protected]>
(cherry picked from commit 87e34a3)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@codecov
Copy link

codecov bot commented Jun 15, 2023

Codecov Report

Merging #8075 (628ae23) into 2.x (b642e69) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                2.x    #8075      +/-   ##
============================================
- Coverage     70.68%   70.67%   -0.01%     
+ Complexity    56849    56818      -31     
============================================
  Files          4702     4702              
  Lines        268993   268993              
  Branches      39768    39768              
============================================
- Hits         190133   190120      -13     
+ Misses        62606    62564      -42     
- Partials      16254    16309      +55     
Impacted Files Coverage Δ
...rg/opensearch/cluster/routing/WeightedRouting.java 65.51% <0.00%> (ø)

... and 457 files with indirect coverage changes

@kotwanikunal
Copy link
Member

Looks like codecov was overridden in #8057 as well.

@andrross andrross merged commit 933cfdd into 2.x Jun 15, 2023
@github-actions github-actions bot deleted the backport/backport-8057-to-2.x branch June 15, 2023 21:39
@andrross
Copy link
Member

Looks like codecov was overridden in #8057 as well.

There is an integration test here, but codecov doesn't account for that

gaiksaya pushed a commit to gaiksaya/OpenSearch that referenced this pull request Jun 26, 2023
…-project#8057) (opensearch-project#8075)

(cherry picked from commit 87e34a3)

Signed-off-by: Rishab Nahata <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants