-
Notifications
You must be signed in to change notification settings - Fork 634
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
enhance(client): improve the client restore process #2031
base: master
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for cubefs-check ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
8b1ca68
to
2e90299
Compare
ed9fdbf
to
ed5abf7
Compare
Codecov Report
@@ Coverage Diff @@
## master #2031 +/- ##
==========================================
- Coverage 45.27% 44.94% -0.33%
==========================================
Files 448 446 -2
Lines 71837 71805 -32
==========================================
- Hits 32521 32276 -245
- Misses 37056 37292 +236
+ Partials 2260 2237 -23
... and 74 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@leonrayang @Victor1319 Please take a look at this PR. Thanks a lot. |
Signed-off-by: liubingxing <liubbingxing@gmail.com>
Signed-off-by: liubingxing <liubbingxing@gmail.com>
ed5abf7
to
de09687
Compare
@sejust Code conflict resolved, please review again. Thanks a lot. |
The following figure shows the process of upgrading cfs-client. This is a good feature and we want to use it in production environment.
We tested it for a while and found some problems, especially when the old client processing quit but the new client failed to start up, will cause the test failed.
So we want to improve the process that the old client waits until the new client restored data.
The main changes are as follow: