Skip to content
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.

Sign up for GitHub

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

Jump to bottom

configuration: fix resource configuration #1704

Merged
merged 6 commits into from Aug 7, 2019
Merged

configuration: fix resource configuration #1704

merged 6 commits into from Aug 7, 2019

Conversation

xiaojingchen
Copy link
Contributor

What is changed, added or deleted?

this PR adds the resource configuration requirements for NUMA architecture CPU

Checklist

  • No Tab spaces anywhere
  • Leave a blank line both before and after a code block

tennix
tennix reviewed Aug 2, 2019
View reviewed changes
dev/tidb-in-kubernetes/reference/configuration/tidb-cluster.md Outdated

其中 PD/TiKV/TiDB 是 TiDB 集群的核心服务组件，在生产环境下它们的资源配置需要按组件要求指定，具体参考：[资源配置推荐](/how-to/deploy/hardware-recommendations.md)。如果是测试环境，可以无需配置资源直接使用 `values.yaml` 中默认的配置。
其中 PD、TiKV、TiDB 是 TiDB 集群的核心服务组件，在生产环境下它们的资源配置需要按组件要求指定，具体参考：[资源配置推荐](/how-to/deploy/hardware-recommendations.md)。如果使用 NUMA 架构的 CPU，为了获得更好的性能，请务必进行绑核，同时保证 PD、TiKV、TiDB 组件获得 Guaranteed 级别的 QoS，具体要求在配置资源时 limits 要等于 requests，并且 CPU 的数量必须是大于或等于 1 的整数。这块请参考: [Guaranteed](https://kubernetes.io/docs/tasks/administer-cluster/cpu-management-policies/#static-policy)。
Copy link
Member

Choose a reason for hiding this comment

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

绑核就是设置 Guaranteed QoS，并且 CPU 为整数，绑核与 Guaranteed QoS 是不是平行的概念。

@xiaojingchen
Copy link
Contributor Author

@tennix PTAL again

dcalvin
dcalvin approved these changes Aug 5, 2019
View reviewed changes
Copy link
Contributor

@dcalvin dcalvin left a comment

Choose a reason for hiding this comment

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

LGTM

tennix
tennix approved these changes Aug 5, 2019
View reviewed changes
Copy link
Member

@tennix tennix left a comment

Choose a reason for hiding this comment

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

LGTM

@lilin90 lilin90 merged commit c334aee into pingcap:master Aug 7, 2019
@lilin90 lilin90 added the translation/doing This PR’s assignee is translating this PR. label Aug 7, 2019
This was referenced Aug 7, 2019
Closed
Merged
@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 Aug 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dcalvin dcalvin dcalvin approved these changes

@tennix tennix tennix approved these changes

Assignees

@junlan-zhang junlan-zhang

Labels
translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@xiaojingchen @dcalvin @tennix @lilin90 @junlan-zhang