Skip to content

master: add new resource-control TiKV configs#23035

Open
mittalrishabh wants to merge 3 commits into
pingcap:masterfrom
mittalrishabh:tikv-8.5-resource-control-configs
Open

master: add new resource-control TiKV configs#23035
mittalrishabh wants to merge 3 commits into
pingcap:masterfrom
mittalrishabh:tikv-8.5-resource-control-configs

Conversation

@mittalrishabh

@mittalrishabh mittalrishabh commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Add 11 new configuration parameters to the resource-control section introduced by TiKV PRs cherry-picked into the v8.5.7 release:

  • bg-cpu-throttle-threshold / fg-cpu-throttle-threshold (CPU throttling)
  • bg-compaction-pressure-threshold / bg-write-io-ceiling / bg-write-io-floor (I/O throttling)
  • enable-fair-scheduling / enable-read-admission-control / enable-write-admission-control
  • historical-usage-window-mins / baseline-burst-pct / admission-max-delayed-count

First-time contributors' checklist

What is changed, added or deleted? (Required)

Which TiDB version(s) do your changes apply to? (Required)

Tips for choosing the affected version(s):

By default, CHOOSE MASTER ONLY so your changes will be applied to the next TiDB major or minor releases. If your PR involves a product feature behavior change or a compatibility change, CHOOSE THE AFFECTED RELEASE BRANCH(ES) AND MASTER.

For details, see tips for choosing the affected versions.

  • master (the latest development version)
  • v9.0 (TiDB 9.0 versions)
  • v8.5 (TiDB 8.5 versions)
  • v8.1 (TiDB 8.1 versions)
  • v7.5 (TiDB 7.5 versions)
  • v7.1 (TiDB 7.1 versions)
  • v6.5 (TiDB 6.5 versions)
  • v6.1 (TiDB 6.1 versions)

What is the related PR or file link(s)?

  • This PR is translated from:
  • Other reference link(s):

Do your changes match any of the following descriptions?

  • Delete files
  • Change aliases
  • Need modification after applied to another branch
  • Might cause conflicts after applied to another branch

@ti-chi-bot ti-chi-bot Bot added contribution This PR is from a community contributor. needs-ok-to-test Indicates a PR created by contributors and need ORG member send '/ok-to-test' to start testing. labels Jun 10, 2026
@ti-chi-bot

ti-chi-bot Bot commented Jun 10, 2026

Copy link
Copy Markdown

Hi @mittalrishabh. Thanks for your PR.

I'm waiting for a pingcap member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

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 kubernetes-sigs/prow repository.

@ti-chi-bot ti-chi-bot Bot added the missing-translation-status This PR does not have translation status info. label Jun 10, 2026
@pingcap-cla-assistant

pingcap-cla-assistant Bot commented Jun 10, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@ti-chi-bot ti-chi-bot Bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jun 10, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds documentation for several new TiKV configuration items introduced in v8.5.6 and v9.0.0, including CPU throttling thresholds, write I/O limits, fair scheduling, and admission control settings. The review feedback suggests style improvements across all these additions to adhere to the repository style guide, specifically by replacing passive voice with active voice, using the second person ("you") when addressing users, and standardizing formatting.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread tikv-configuration-file.md Outdated
Comment thread tikv-configuration-file.md Outdated
Comment thread tikv-configuration-file.md Outdated
Comment thread tikv-configuration-file.md Outdated
Comment thread tikv-configuration-file.md
Comment thread tikv-configuration-file.md Outdated
Comment thread tikv-configuration-file.md Outdated
Comment thread tikv-configuration-file.md Outdated
Comment thread tikv-configuration-file.md Outdated
Comment thread tikv-configuration-file.md Outdated
@lilin90 lilin90 changed the title tikv config: document new resource-control configs for v8.5.6 master: add new resource-control TiKV configs Jun 26, 2026
@lilin90 lilin90 self-assigned this Jun 26, 2026
@lilin90 lilin90 added the translation/doing This PR's assignee is translating this PR. label Jun 26, 2026
@ti-chi-bot ti-chi-bot Bot removed the missing-translation-status This PR does not have translation status info. label Jun 26, 2026
@lilin90 lilin90 added type/compatibility-or-feature-change This PR involves compatibility changes or feature behavior changes. needs-cherry-pick-release-8.5 Should cherry pick this PR to release-8.5 branch. labels Jun 26, 2026
Comment thread tikv-configuration-file.md Outdated
Comment thread tikv-configuration-file.md Outdated
Comment thread tikv-configuration-file.md Outdated
Comment thread tikv-configuration-file.md Outdated
Comment thread tikv-configuration-file.md Outdated
Comment thread tikv-configuration-file.md Outdated
@ti-chi-bot

ti-chi-bot Bot commented Jun 26, 2026

Copy link
Copy Markdown

[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 ask for approval from lilin90. For more information see the Code Review Process.
Please ensure that each of them provides their approval before proceeding.

The full list of commands accepted by this bot can be found 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

@lilin90

lilin90 commented Jun 29, 2026

Copy link
Copy Markdown
Member

/ok-to-test

@ti-chi-bot ti-chi-bot Bot added ok-to-test Indicates a PR is ready to be tested. and removed needs-ok-to-test Indicates a PR created by contributors and need ORG member send '/ok-to-test' to start testing. labels Jun 29, 2026
@lilin90

lilin90 commented Jun 29, 2026

Copy link
Copy Markdown
Member

CLA assistant check Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.✅ lilin90✅ mittalrishabh❌ rishabh mittal

rishabh mittal seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @mittalrishabh, it looks like the CLA bot cannot associate the email address used in your commit with a GitHub account. Related commits: 1e3ffcb, eebd83f.

Could you please check that the email address used for your commit has been added and verified in your GitHub account (Settings → Emails)? If the commit was created with a different email address, please amend the commit with the correct email and force-push the updated commit. The CLA check should then pass automatically.

Signed-off-by: rishabh mittal <mittalrishabh@gmail.com>
@mittalrishabh mittalrishabh force-pushed the tikv-8.5-resource-control-configs branch from d30f984 to 201863d Compare June 30, 2026 23:52

@lilin90 lilin90 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update anchor links

Comment thread tikv-configuration-file.md Outdated
Comment thread tikv-configuration-file.md Outdated
Comment thread tikv-configuration-file.md Outdated
Comment thread tikv-configuration-file.md Outdated
Comment thread tikv-configuration-file.md Outdated
Comment thread tikv-configuration-file.md Outdated
@lilin90 lilin90 added the requires-version-specific-changes After cherry-picked, the cherry-picked PR requires further changes. label Jul 1, 2026
github-actions Bot added a commit to lilin90/docs-cn that referenced this pull request Jul 1, 2026
Synced from: pingcap/docs#23035
Target PR: pingcap#21754
AI Provider: azure

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@lilin90 lilin90 requested a review from v01dstar July 1, 2026 07:15
@lilin90 lilin90 added translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. and removed translation/doing This PR's assignee is translating this PR. labels Jul 1, 2026
Comment thread tikv-configuration-file.md Outdated

### `historical-usage-window-mins` <span class="version-mark">New in v8.5.7 and v9.0.0</span>

+ Specifies the size of the sliding time window (in minutes) that TiKV uses to compute per-resource-group historical RU baselines. A larger window smooths out short-term bursts, while a smaller window makes the baseline more responsive to recent usage. Valid range: `2-60`. **You must restart TiKV for changes to this configuration to take effect.**

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just realized that we don't validate the config in the code, although it says valid range is 2-60. I think we can merge this doc. And the validation in a PR later.

@v01dstar v01dstar left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@ti-chi-bot

ti-chi-bot Bot commented Jul 3, 2026

Copy link
Copy Markdown

@v01dstar: adding LGTM is restricted to approvers and reviewers in OWNERS files.

Details

In response to this:

lgtm

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 kubernetes-sigs/prow repository.


### `enable-write-admission-control` <span class="version-mark">New in v8.5.7 and v9.0.0</span>

+ Controls whether to enable admission control for write requests. When enabled, write requests from over-baseline resource groups are delayed or rejected with `SchedTooBusy` when CPU utilization exceeds [`fg-cpu-throttle-threshold`](#fg-cpu-throttle-threshold-new-in-v857-and-v900). This configuration takes effect only when [`enable-fair-scheduling`](#enable-fair-scheduling-new-in-v857-and-v900) is also enabled.

@v01dstar v01dstar Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This configuration takes effect only when enable-fair-scheduling is also enabled.

AI thinks this is not the case. The test test_admission_decision() is the evidence.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution This PR is from a community contributor. needs-cherry-pick-release-8.5 Should cherry pick this PR to release-8.5 branch. ok-to-test Indicates a PR is ready to be tested. requires-version-specific-changes After cherry-picked, the cherry-picked PR requires further changes. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. type/compatibility-or-feature-change This PR involves compatibility changes or feature behavior changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants