tiup: complete tiup cluster tls desc (#22861)#22873
Conversation
Synced from: pingcap/docs-cn#21612 Target PR: pingcap#22861 AI Provider: azure Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
ti-chi-bot
added
lgtm
size/XS
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request adds a note to the tiup cluster tls documentation explaining that the command currently only supports clusters with a single PD node. The review feedback suggests formatting the multi-step workaround as an ordered list to align with the style guide and simplifying the technical language for better readability.
| > **Note:** | ||
| > | ||
| > Currently, the `tiup cluster tls` command supports enabling or disabling TLS only for clusters with a single PD node. For clusters with multiple PD nodes, executing the `tiup cluster tls` command directly returns an error, because switching the TLS status can cause communication exceptions between PD nodes. To enable or disable TLS for a cluster with multiple PD nodes, first [`scale-in`](/tiup/tiup-component-cluster-scale-in.md) the PD nodes to one node, and then execute the `tiup cluster tls` command. |
There was a problem hiding this comment.
According to the style guide, steps should be presented as an ordered list for better readability. I have also simplified the wording for clarity and to avoid technical jargon like "communication exceptions".
| > **Note:** | |
| > | |
| > Currently, the `tiup cluster tls` command supports enabling or disabling TLS only for clusters with a single PD node. For clusters with multiple PD nodes, executing the `tiup cluster tls` command directly returns an error, because switching the TLS status can cause communication exceptions between PD nodes. To enable or disable TLS for a cluster with multiple PD nodes, first [`scale-in`](/tiup/tiup-component-cluster-scale-in.md) the PD nodes to one node, and then execute the `tiup cluster tls` command. | |
| > **Note:** | |
| > | |
| > Currently, the `tiup cluster tls` command only supports enabling or disabling TLS for clusters with a single PD node. If a cluster has multiple PD nodes, the command fails because switching the TLS status might cause communication issues between PD nodes. | |
| > | |
| > To enable or disable TLS for a cluster with multiple PD nodes: | |
| > | |
| > 1. [`scale-in`](/tiup/tiup-component-cluster-scale-in.md) the PD nodes to a single node. | |
| > 2. Run the `tiup cluster tls` command. |
2 participants
This is an automated cherry-pick of #22861
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 (in Chinese).
What is the related PR or file link(s)?
Do your changes match any of the following descriptions?