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.

Sign up for GitHub

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

Jump to bottom

[10.0] Update healthy tablets correctly for primary down #7937

Merged
merged 2 commits into from
Apr 23, 2021
Merged

[10.0] Update healthy tablets correctly for primary down #7937

merged 2 commits into from
Apr 23, 2021

Conversation

deepthi
Copy link
Member

@deepthi deepthi commented Apr 22, 2021

Description

Cherry-pick of #7915

Related Issue(s)

#7906

Checklist

  • Should this PR be backported?
  • Tests were added or are not required
  • Documentation was added or is not required

Deployment Notes

Impacted Areas in Vitess

Components that this PR will affect:

  • Query Serving
  • VReplication
  • Cluster Management
  • Build/CI
  • VTAdmin

@5antelope @deepthi
Check tablet alias before removing after error stream
0353021 
Signed-off-by: crowu <y.wu4515@gmail.com>
@5antelope @deepthi
Only modify healthy map
a7cfe60 
Signed-off-by: crowu <y.wu4515@gmail.com>
@deepthi deepthi requested a review from askdba April 22, 2021 20:02
@deepthi deepthi changed the title [10.0]: Update healthy tablets correctly for primary down [10.0] Update healthy tablets correctly for primary down Apr 22, 2021
@askdba askdba merged commit 5c07e03 into vitessio:release-10.0 Apr 23, 2021
@askdba askdba deleted the ds-backport-7915 branch April 23, 2021 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@askdba askdba askdba approved these changes

Assignees
No one assigned
Labels
Component: Query Serving
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@deepthi @askdba @5antelope