*: report unhealthy on the status port when the memory usage is too high - #1212
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1212 +/- ##
=======================================
Coverage ? 67.71%
=======================================
Files ? 152
Lines ? 16915
Branches ? 0
=======================================
Hits ? 11454
Misses ? 4692
Partials ? 769
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
/retest |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: YangKeao The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
[LGTM Timeline notifier]Timeline:
|
|
/cherry-pick release-1.3 |
|
@djshow832: The label(s) DetailsIn response to this:
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. |
|
@djshow832: new pull request created to branch DetailsIn response to this:
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. |
Resolve conflicts for release-1.3: drop the meter field, the manual health override, the ConfigManager interface, and the client-mode replayer cmd that are absent on this branch. Add UnhealthyReason to HealthInfo so DebugHealth can report the reason. Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io> Co-authored-by: Cursor <cursoragent@cursor.com>
|
/cherry-pick release-nextgen-202603 |
|
@djshow832: The label(s) DetailsIn response to this:
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. |
|
@djshow832: new pull request created to branch DetailsIn response to this:
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. |
What problem does this PR solve?
Issue Number: close #1119
Problem Summary:
#1120 only rejects connections when the memory is too high, but it doesn't report unhealthy on the status port.
What is changed and how it works:
Check List
Tests
Notable changes
Release note
Please refer to Release Notes Language Style Guide to write a quality release note.