Closed xhebox closed 3 months ago
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please assign kaaaaaaang for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.
Attention: 34 lines
in your changes are missing coverage. Please review.
Comparison is base (
271d5ea
) 50.19% compared to head (6059a2d
) 55.30%.:exclamation: Current head 6059a2d differs from pull request most recent head c0e1bee. Consider uploading reports for the commit c0e1bee to get more accurate results
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
What problem does this PR solve?
User needs to manually set
session-xxxxx
for tidb, to make tiproxy migration works. However, we could just generate a self-signed cert automatically. There is no security implication: the only requirement for this cert, is that, it should be same on all tidb instances.Check List
Tests
Code changes
Side effects
Related changes
Release notes: