Skip to content

tiproxy: delete drained pods early when the connection drops to zero - #6936

Merged
ti-chi-bot[bot] merged 3 commits into
pingcap:mainfrom
YangKeao:close-tiproxy-after-connection-drops-to-zero
Jun 15, 2026
Merged

tiproxy: delete drained pods early when the connection drops to zero#6936
ti-chi-bot[bot] merged 3 commits into
pingcap:mainfrom
YangKeao:close-tiproxy-after-connection-drops-to-zero

Conversation

@YangKeao

Copy link
Copy Markdown
Member

If the connection on a specific tiproxy pod has already dropped to zero, we can delete it earlier. It'll optimize the total time consumed by rolling update / scaling in.

@github-actions github-actions Bot added the v2 for operator v2 label Jun 11, 2026
@YangKeao
YangKeao marked this pull request as draft June 11, 2026 06:41
@codecov-commenter

codecov-commenter commented Jun 11, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 79.71014% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 39.40%. Comparing base (6941259) to head (ca4ecd7).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6936      +/-   ##
==========================================
+ Coverage   39.30%   39.40%   +0.10%     
==========================================
  Files         430      430              
  Lines       24310    24376      +66     
==========================================
+ Hits         9554     9606      +52     
- Misses      14756    14770      +14     
Flag Coverage Δ
unittest 39.40% <79.71%> (+0.10%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@YangKeao
YangKeao marked this pull request as ready for review June 11, 2026 07:03
@ti-chi-bot
ti-chi-bot Bot requested a review from shonge June 11, 2026 07:03
@YangKeao
YangKeao requested a review from liubog2008 June 11, 2026 07:59
@YangKeao

Copy link
Copy Markdown
Member Author

/hold

@YangKeao

Copy link
Copy Markdown
Member Author

/retest

@YangKeao
YangKeao marked this pull request as ready for review June 11, 2026 16:23
@YangKeao

Copy link
Copy Markdown
Member Author

/retest

@YangKeao

Copy link
Copy Markdown
Member Author

/retest

1 similar comment
@YangKeao

Copy link
Copy Markdown
Member Author

/retest

@YangKeao

Copy link
Copy Markdown
Member Author

/unhold

@liubog2008

Copy link
Copy Markdown
Member

/lgtm

@ti-chi-bot ti-chi-bot Bot added the lgtm label Jun 15, 2026
@ti-chi-bot

ti-chi-bot Bot commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

[LGTM Timeline notifier]

Timeline:

  • 2026-06-15 09:55:47.859095231 +0000 UTC m=+1385848.929412621: ☑️ agreed by liubog2008.

@ti-chi-bot

ti-chi-bot Bot commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: liubog2008

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ti-chi-bot ti-chi-bot Bot added the approved label Jun 15, 2026
@ti-chi-bot
ti-chi-bot Bot merged commit 28394a5 into pingcap:main Jun 15, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants