-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
upgrade pd-client to support pd client tls reload #15071
Conversation
need wait pingcap/tidb-tools#320 merge |
Codecov Report
@@ Coverage Diff @@
## master #15071 +/- ##
================================================
- Coverage 80.4574% 80.3279% -0.1296%
================================================
Files 503 503
Lines 133693 132777 -916
================================================
- Hits 107566 106657 -909
+ Misses 17752 17723 -29
- Partials 8375 8397 +22 |
due to go-sql-driver/mysql#302 we need retry "invalid connection" https://github.com/pingcap/tidb/pull/15071/files#diff-3b19a7bb39c0907c6c0615c217528259R360 |
/run-all-tests |
/run-unit-test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/merge |
/run-all-tests |
cherry pick to release-3.0 in PR #15243 |
What problem does this PR solve?
upgrade pd-client to newest version to support reload cluster-tls
also upgrade tidb-tools which use pd-client too(pingcap/tidb-tools#320 but it meets some CI problem due to pd's new go mod version)
What is changed and how it works?
upgrade dep
Check List
Tests
Code changes
Side effects
Related changes
Release note
This change is