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] Do not fail replica shard due to primary closure #4309

Merged
merged 2 commits into from
Aug 27, 2022

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 1dbb63a from #4133

This commit prevents a replica shard from being failed in the case that
a replication action to a replica is canceled due to the primary shard
being closed.

Signed-off-by: Andrew Ross <andrross@amazon.com>

Signed-off-by: Andrew Ross <andrross@amazon.com>
(cherry picked from commit 1dbb63a)
@opensearch-trigger-bot opensearch-trigger-bot bot requested review from a team and reta as code owners August 26, 2022 19:59
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@codecov-commenter
Copy link

codecov-commenter commented Aug 26, 2022

Codecov Report

Merging #4309 (ff1e80d) into 2.x (318ec03) will decrease coverage by 0.01%.
The diff coverage is 83.33%.

@@             Coverage Diff              @@
##                2.x    #4309      +/-   ##
============================================
- Coverage     70.66%   70.65%   -0.02%     
+ Complexity    56932    56924       -8     
============================================
  Files          4570     4571       +1     
  Lines        273527   273530       +3     
  Branches      40110    40111       +1     
============================================
- Hits         193301   193266      -35     
+ Misses        64071    64069       -2     
- Partials      16155    16195      +40     
Impacted Files Coverage Δ
...earch/index/shard/PrimaryShardClosedException.java 50.00% <50.00%> (ø)
.../main/java/org/opensearch/OpenSearchException.java 92.22% <100.00%> (-1.22%) ⬇️
...support/replication/PendingReplicationActions.java 90.47% <100.00%> (-0.64%) ⬇️
...tion/support/replication/TransportWriteAction.java 81.34% <100.00%> (+0.89%) ⬆️
...adcast/BroadcastShardOperationFailedException.java 55.55% <0.00%> (-44.45%) ⬇️
.../admin/cluster/reroute/ClusterRerouteResponse.java 60.00% <0.00%> (-40.00%) ⬇️
...luster/routing/allocation/RoutingExplanations.java 62.06% <0.00%> (-37.94%) ⬇️
...java/org/opensearch/threadpool/ThreadPoolInfo.java 56.25% <0.00%> (-37.50%) ⬇️
...ion/admin/cluster/node/info/PluginsAndModules.java 53.12% <0.00%> (-34.38%) ⬇️
...ensearch/client/indices/DetailAnalyzeResponse.java 20.54% <0.00%> (-34.25%) ⬇️
... and 471 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Signed-off-by: Andrew Ross <andrross@amazon.com>
@andrross andrross force-pushed the backport/backport-4133-to-2.x branch from ed2a606 to ff1e80d Compare August 26, 2022 22:54
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@andrross
Copy link
Member

andrross commented Aug 26, 2022

Failure in SegmentReplicationTargetServiceTests. Can't reproduce locally, looks like a flaky test. Issue exists here: #4268

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@dreamer-89
Copy link
Member

Failure in SegmentReplicationTargetServiceTests. Can't reproduce locally, looks like a flaky test. Issue exists here: #4268

Created #4312 for tracking this issue.

@dreamer-89 dreamer-89 added the v2.3.0 'Issues and PRs related to version v2.3.0' label Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v2.3.0 'Issues and PRs related to version v2.3.0'
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants