Skip to content

slack-19.0: backport 16572 (bug fix for incorrect order of steps in SwitchTraffic)#702

Merged
tanjinx merged 2 commits intoslack-19.0from
tanjin-backport-16572
Aug 28, 2025
Merged

slack-19.0: backport 16572 (bug fix for incorrect order of steps in SwitchTraffic)#702
tanjinx merged 2 commits intoslack-19.0from
tanjin-backport-16572

Conversation

@tanjinx
Copy link

@tanjinx tanjinx commented Aug 28, 2025

Description

backport vitessio#16572 (vreplication bug)

VReplication: Gather source positions once we know all writes are done during traffic switch vitessio#16572
BugReport: Incorrect order of steps in SwitchTraffic vitessio#16563

Related Issue(s)

Checklist

  • "Backport to:" labels have been added if this change should be back-ported to release branches
  • If this change is to be back-ported to previous releases, a justification is included in the PR description
  • Tests were added or are not required
  • Did the new or modified tests pass consistently locally and on CI?
  • Documentation was added or is not required

Deployment Notes

mattlord and others added 2 commits August 28, 2025 10:17
…e during traffic switch (vitessio#16572)

Signed-off-by: Matt Lord <mattalord@gmail.com>
Signed-off-by: Tanjin Xu <tanjin.xu@slack-corp.com>
Signed-off-by: Tanjin Xu <tanjin.xu@slack-corp.com>
@salesforce-cla
Copy link

Thanks for the contribution! Before we can merge this, we need @mattlord to sign the Salesforce Inc. Contributor License Agreement.

@github-actions github-actions bot added this to the v19.0.7 milestone Aug 28, 2025
@tanjinx tanjinx marked this pull request as ready for review August 28, 2025 18:35
@tanjinx tanjinx requested a review from a team as a code owner August 28, 2025 18:35
@tanjinx tanjinx merged commit 68ad8a1 into slack-19.0 Aug 28, 2025
161 of 170 checks passed
@tanjinx tanjinx deleted the tanjin-backport-16572 branch August 28, 2025 19:35
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.

3 participants