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.

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

tidb-in-kubernetes: add permitHost document #1777

Merged
merged 20 commits into from
Oct 28, 2019
Merged

Conversation

shonge
Copy link
Member

@shonge shonge commented Aug 19, 2019

What is changed, added or deleted?

tidb-in-kubernetes document: add permithost document.

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

pingcap/tidb-operator#733

Which version does your change affect?

tidb-operator v1.0 +

@CLAassistant
Copy link

CLAassistant commented Aug 19, 2019

CLA assistant check
All committers have signed the CLA.

@yikeke yikeke requested a review from DanielZhangQD August 20, 2019 03:10
@yikeke yikeke added contribution This PR is from a community contributor. tidb-in-k8s labels Aug 20, 2019
2. 部署集群
2. 设置允许访问 TiDB 的主机

在部署集群前可以通过设置 `permitHost` 设置允许访问 TiDB 的主机 **host_name**。如果不设置,则允许所有主机访问。详情请参考 [Mysql GRANT host name](https://dev.mysql.com/doc/refman/5.7/en/grant.html)。
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
在部署集群前可以通过设置 `permitHost` 设置允许访问 TiDB 的主机 **host_name**。如果不设置,则允许所有主机访问。详情请参考 [Mysql GRANT host name](https://dev.mysql.com/doc/refman/5.7/en/grant.html)。
在部署集群前可以通过设置 `tidb.permitHost` 设置允许访问 TiDB 的主机 **host_name**。如果不设置,则允许所有主机访问。详情请参考 [Mysql GRANT host name](https://dev.mysql.com/doc/refman/5.7/en/grant.html)。

@pingcap pingcap deleted a comment from DanielZhangQD Aug 22, 2019
@weekface
Copy link
Contributor

Please do not merge this pr before: pingcap/tidb-operator#779

@yikeke yikeke added DNM translation/doing This PR’s assignee is translating this PR. labels Aug 26, 2019
@lilin90 lilin90 changed the title add permitHost document tidb-in-kubernetes: add permitHost document Aug 28, 2019
@@ -33,9 +33,19 @@ category: how-to
kubectl create secret generic tidb-secret --from-literal=root=<root-password> --from-literal=developer=<developer-passowrd> --namespace=<namespace>
```

该命令会创建 `root` 和 `developer` 两个用户的密码,存到 `tidb-secret` 的 Secret 里面。
该命令会创建 `root` 和 `developer` 两个用户的密码,存到 `tidb-secret` 的 Secret 里面。并且创建的普通用户`developer`默认只有`USAGE`权限,其他权限请在`tidb.initSql`中设置。
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
该命令会创建 `root` 和 `developer` 两个用户的密码,存到 `tidb-secret` 的 Secret 里面。并且创建的普通用户`developer`默认只有`USAGE`权限,其他权限请在`tidb.initSql`中设置。
该命令会创建 `root` 和 `developer` 两个用户的密码,存到 `tidb-secret` 的 Secret 里面。并且创建的普通用户 `developer` 默认只有 `USAGE` 权限,其他权限请在 `tidb.initSql` 中设置。

@@ -34,9 +34,19 @@ aliases: ['/docs-cn/v3.0/how-to/configure/initialize-cluster/']
kubectl create secret generic tidb-secret --from-literal=root=<root-password> --from-literal=developer=<developer-passowrd> --namespace=<namespace>
```

该命令会创建 `root` 和 `developer` 两个用户的密码,存到 `tidb-secret` 的 Secret 里面。
该命令会创建 `root` 和 `developer` 两个用户的密码,存到 `tidb-secret` 的 Secret 里面。并且创建的普通用户developer默认只有USAGE权限,其他权限请在tidb.initSql中设置。
Copy link
Member

Choose a reason for hiding this comment

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

  • Keep the format consistent.
  • Leave a space between Chinese and English characters.
Suggested change
该命令会创建 `root` 和 `developer` 两个用户的密码,存到 `tidb-secret` 的 Secret 里面。并且创建的普通用户developer默认只有USAGE权限,其他权限请在tidb.initSql中设置
该命令会创建 `root` 和 `developer` 两个用户的密码,存到 `tidb-secret` 的 Secret 里面。并且创建的普通用户 `developer` 默认只有 `USAGE` 权限,其他权限请在 `tidb.initSql` 中设置

Copy link
Contributor

@DanielZhangQD DanielZhangQD left a comment

Choose a reason for hiding this comment

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

Please address the comments from @lilin90, LGTM for the rest.

该命令会创建 `root` 和 `developer` 两个用户的密码,存到 `tidb-secret` 的 Secret 里面。
该命令会创建 `root` 和 `developer` 两个用户的密码,存到 `tidb-secret` 的 Secret 里面。并且创建的普通用户developer默认只有USAGE权限,其他权限请在tidb.initSql中设置。

2. 设置允许访问 TiDB 的主机
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
2. 设置允许访问 TiDB 的主机
3. 设置允许访问 TiDB 的主机

tidb:
passwordSecretName: tidb-secret
permitHost: <Mysql-client-host-name>
```

3. 部署集群
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
3. 部署集群
4. 部署集群

@lilin90 lilin90 removed the DNM label Oct 28, 2019
Copy link
Member

@lilin90 lilin90 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 b46709a into pingcap:master Oct 28, 2019
@junlan-zhang junlan-zhang 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 Oct 29, 2019
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. translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants