Closed xhebox closed 9 months ago
Attention: 16 lines
in your changes are missing coverage. Please review.
Comparison is base (
48b5119
) 55.42% compared to head (4145baf
) 55.05%. Report is 1 commits behind head on master.:exclamation: Current head 4145baf differs from pull request most recent head 4df235a. Consider uploading reports for the commit 4df235a to get more accurate results
Files | Patch % | Lines |
---|---|---|
components/playground/instance/tiproxy.go | 0.00% | 16 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: kaaaaaaang
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/lgtm
Timeline:
2023-12-28 09:10:40.996061608 +0000 UTC m=+1729732.033288530
: :ballot_box_with_check: agreed by kaaaaaaang.2023-12-29 07:22:51.806774186 +0000 UTC m=+1809662.844001113
: :heavy_multiplication_x::repeat: reset by ti-chi-bot[bot].2023-12-29 07:23:57.731474982 +0000 UTC m=+1809728.768701913
: :ballot_box_with_check: agreed by kaaaaaaang.
What problem does this PR solve?
Close #2339
previously, it doesnt override user config correctly leading to port conflicts when using custom configs for multiple tiproxy instances. TiDB pass ports by cli flags so then are fine.
Also, log-file is not passed
What is changed and how it works?
Check List
Tests
Code changes
Side effects
Related changes
Release notes: