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

Fix the race between PromoteReplica and replication manager tick (#9859) - release 12 #10076

Merged
merged 2 commits into from
Apr 14, 2022

Conversation

GuptaManan100
Copy link
Member

Description

Backport #9859

Related Issue(s)

Checklist

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

Deployment Notes

…essio#9859)

* test: improve test to check that replication manager is stopped before we make any changes in MySQL

Signed-off-by: Manan Gupta <manan@planetscale.com>

* feat: stop the replication manager first in PromoteReplica call

Signed-off-by: Manan Gupta <manan@planetscale.com>

* docs: add comments to replication manager setTabletType

Signed-off-by: Manan Gupta <manan@planetscale.com>

* docs: add comment for SetReplicationSource in wrangler

Signed-off-by: Manan Gupta <manan@planetscale.com>

* docs: add comments to SetReplicationSource in reparentutil

Signed-off-by: Manan Gupta <manan@planetscale.com>
@GuptaManan100 GuptaManan100 changed the title Fix the race between PromoteReplica and replication manager tick (#9859) Fix the race between PromoteReplica and replication manager tick (#9859) - release 12 Apr 12, 2022
…r-race-12

Signed-off-by: Manan Gupta <manan@planetscale.com>
@GuptaManan100 GuptaManan100 merged commit e87dfc4 into vitessio:release-12.0 Apr 14, 2022
@GuptaManan100 GuptaManan100 deleted the repl-manager-race-12 branch April 14, 2022 05:39
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