-
Notifications
You must be signed in to change notification settings - Fork 751
mcs: fix tls setting for microservices (#9344) #10005
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
mcs: fix tls setting for microservices (#9344) #10005
Conversation
close tikv#9367 Signed-off-by: ti-chi-bot <[email protected]>
|
@rleungx This PR has conflicts, I have hold it. |
|
@ti-chi-bot: ## If you want to know how to resolve it, please read the guide in TiDB Dev Guide. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the ti-community-infra/tichi repository. |
Signed-off-by: Ryan Leung <[email protected]>
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## release-8.5 #10005 +/- ##
===============================================
+ Coverage 77.74% 77.82% +0.08%
===============================================
Files 467 468 +1
Lines 62474 63144 +670
===============================================
+ Hits 48570 49142 +572
- Misses 10315 10387 +72
- Partials 3589 3615 +26
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
/hold cancel |
[LGTM Timeline notifier]Timeline:
|
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bufferflies, JmPotato, niubell The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This is an automated cherry-pick of #9344
What problem does this PR solve?
Issue Number: Close #9367
What is changed and how does it work?
Check List
Tests
This PR has been validated by operator.
Release note