Repository navigation
Conversation
A dump stream cut between statements, or where the rest is still valid SQL, makes mysql exit 0 and leaves a table missing or short of rows, and the guide's checks could miss that. Add a Common Issues entry with recovery steps, and a per-table COUNT(*) comparison on every member in Step 5. Stop comparing TABLE_ROWS, which is only an estimate for InnoDB. Tested on Percona Server 8.0.39 with a truncated dump and a kubectl test script. Not tested with a real network failure, on 5.7, PXC or a real group. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
|
/test doc-pr-build-mysql-mgr |
Deploying alauda-mysql-mgr with
|
| Latest commit: |
1a2a9a8
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://1e7571ae.alauda-mysql-mgr.pages.dev |
| Branch Preview URL: | https://docs-pxc-migration-network-i-4aan.alauda-mysql-mgr.pages.dev |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
The guide doesn't say what to do if the network breaks during a migration, and its checks can miss the result.
The Step 4 dump is streamed through two
kubectl execsessions. When a dump was cut off in a test,mysqlexited with status 0 and reported no error:mysqlexit codeINSERT, where the remaining text is still valid SQLThe checks in the guide don't catch this reliably: the Step 5 command only compares table counts, and the Troubleshooting script compares
TABLE_ROWS, which is only an estimate for InnoDB.Changes
COUNT(*)comparison on every target member, printingOKorMISMATCH, and say what each result means.TABLE_ROWScomparison with a pointer to Step 5.Verification
Tested on throwaway Percona Server 8.0.39 servers. Every runnable bash block from Getting Started through Step 6 ran in order under
set -uon all three branches, with akubectltest script pointing at the local servers.doom lint2.10.0, the version CI installs, passes on all three.INSERTOKlines (2 tables, 3 members)MISMATCHlinesMISMATCH, 6OKCREATE USERrun twiceERROR 1396Not tested:
kubectl exec. The tests truncated a dump file instead, so thatkubectlreports a broken connection, whichFAILEDrelies on, is expected but unmeasured.tmux, proxies and load balancers, which is general guidance.🤖 Generated with Claude Code