Skip to content

byoc: add onboarding docs and diagram#23256

Open
lilin90 wants to merge 8 commits into
pingcap:release-8.5from
lilin90:byoc-onboarding
Open

byoc: add onboarding docs and diagram#23256
lilin90 wants to merge 8 commits into
pingcap:release-8.5from
lilin90:byoc-onboarding

Conversation

@lilin90

@lilin90 lilin90 commented Jul 13, 2026

Copy link
Copy Markdown
Member

What is changed, added or deleted? (Required)

  • Added TiDB Cloud BYOC onboarding docs
  • Added BYOC architecture diagram

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)

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

@lilin90 lilin90 requested a review from wildpcww July 13, 2026 10:20
@lilin90 lilin90 self-assigned this Jul 13, 2026
@lilin90 lilin90 added translation/no-need No need to translate this PR. BYOC Related to TiDB Cloud BYOC docs. labels Jul 13, 2026
@ti-chi-bot

ti-chi-bot Bot commented Jul 13, 2026

Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@ti-chi-bot ti-chi-bot Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 13, 2026
@ti-chi-bot

ti-chi-bot Bot commented Jul 13, 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

@ti-chi-bot ti-chi-bot Bot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Jul 13, 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 introduces comprehensive documentation for the TiDB Cloud BYOC (Bring Your Own Cloud) onboarding and deployment process on AWS, covering prerequisites, IAM setup, instance creation, joint validation, multi-region deployment, and security hardening. The reviewer's feedback focuses on improving readability and ensuring strict adherence to the style guide. Key recommendations include converting headings to sentence case, correcting typos and broken backticks, replacing full-width punctuation with standard half-width characters, fixing incorrect step/section references, and formatting tables and code blocks for better clarity.

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 tidb-cloud/byoc/automated-deployment.md Outdated

**Note:** This phase is fully managed by TiDB Cloud. No customer action is required until you receive the completion notification.

## **3.1 Deployment Timeline**

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.

low

According to the style guide, headings should use sentence case and avoid unnecessary bold formatting.

Suggested change
## **3.1 Deployment Timeline**
## 3.1 Deployment timeline
References
  1. Use sentence case for headings (e.g., ## Configure the cluster). (link)

Comment thread tidb-cloud/byoc/automated-deployment.md Outdated

* Deploy Data Plane Management Nodes: Hosts components (such as the TiDB Operator) to provide the runtime environment for the subsequent creation of TiDB compute and storage nodes.

## **3.2 Deployment completion**

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.

low

According to the style guide, headings should use sentence case and avoid unnecessary bold formatting.

Suggested change
## **3.2 Deployment completion**
## 3.2 Deployment completion
References
  1. Use sentence case for headings (e.g., ## Configure the cluster). (link)

Comment thread tidb-cloud/byoc/automated-deployment.md Outdated

Once the automation completes:

1. **Notification: You will be notified by the TiDB Cloud Team that the BYOC Region is ready.**

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.

low

Avoid passive voice and unnecessary bolding of the entire list item. Also, 'team' and 'region' should be lowercase.

Suggested change
1. **Notification: You will be notified by the TiDB Cloud Team that the BYOC Region is ready.**
1. **Notification:** The TiDB Cloud team will notify you when the BYOC region is ready.
References
  1. Avoid passive voice and use consistent lowercase terminology for non-proper nouns. (link)

| Note: Each phase requires timely coordination between teams. We recommend assigning dedicated resources from both organizations to ensure smooth progression through all stages. |
| :---- |

## **Architecture Overview**

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.

low

According to the style guide, headings should use sentence case and avoid unnecessary bold formatting.

Suggested change
## **Architecture Overview**
## Architecture overview
References
  1. Use sentence case for headings (e.g., ## Configure the cluster). (link)


1. Log in to the [TiDB Cloud Console](https://tidbcloud.com/).

If you do not have a TiDB Cloud account, click [here](https://tidbcloud.com/free-trial) to sign up.* *You may sign up with an email/password to manage TiDB Cloud credentials, or use single sign-on (SSO) via google, Github, or Microsoft accounts.

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.

low

Fix the typo '* *', use 'can' instead of 'may' per the style guide, and capitalize 'Google' and 'GitHub'.

Suggested change
If you do not have a TiDB Cloud account, click [here](https://tidbcloud.com/free-trial) to sign up.* *You may sign up with an email/password to manage TiDB Cloud credentials, or use single sign-on (SSO) via google, Github, or Microsoft accounts.
If you do not have a TiDB Cloud account, click [here](https://tidbcloud.com/free-trial) to sign up. You can sign up with an email/password to manage TiDB Cloud credentials, or use single sign-on (SSO) via Google, GitHub, or Microsoft accounts.
References
  1. Correct English grammar, spelling, and punctuation mistakes. (link)


* *Initial Setup Time:* The creation of the **first instance** typically takes approximately **1 hour** as the system initializes the Kubernetes environment.

* *Subsequentinstances:* Creating additional instances in the same region will only take a few minutes.

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.

low

Fix the typo 'Subsequentinstances' by adding a space.

Suggested change
* *Subsequentinstances:* Creating additional instances in the same region will only take a few minutes.
* *Subsequent instances:* Creating additional instances in the same region will only take a few minutes.
References
  1. Correct English grammar, spelling, and punctuation mistakes. (link)

Comment thread tidb-cloud/byoc/joint-validation.md Outdated
| Category | Validation Item | Owner |
| :---- | :---- | :---- |
| **Connectivity** | SQL Endpoint: verify connectivity to both private and public SQL endpoints from your application servers. | Customer |
| | Secure Tunnel(Optional): Confirm stable VPN links via the Bastion Host. | TiDB Cloud |

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.

low

Replace full-width parentheses with standard half-width parentheses, and use lowercase for 'bastion host'.

Suggested change
| | Secure TunnelOptional: Confirm stable VPN links via the Bastion Host. | TiDB Cloud |
| | Secure Tunnel (Optional): Confirm stable VPN links via the bastion host. | TiDB Cloud |

Comment thread tidb-cloud/byoc/multi-region-deployment.md Outdated
Comment thread tidb-cloud/byoc/multi-region-deployment.md Outdated
Comment thread tidb-cloud/byoc/multi-region-deployment.md
lilin90 added 3 commits July 13, 2026 18:30
Rename and reorganize the BYOC onboarding guide into clearer phase-based pages, update links in the overview, and rewrite deployment, IAM, service initialization, validation, multi-region, and security hardening content for consistency.
@lilin90 lilin90 marked this pull request as ready for review July 14, 2026 08:35
@ti-chi-bot ti-chi-bot Bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BYOC Related to TiDB Cloud BYOC docs. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. translation/no-need No need to translate this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant