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

ZH-trans:add kubeadm_init_phase_certs_front-proxy-client.md #15795

Merged
merged 1 commit into from Aug 11, 2019

Conversation

@markthink
Copy link
Member

left a comment

/lgtm
/approve

@k8s-ci-robot

This comment has been minimized.

Copy link

commented Aug 11, 2019

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: markthink

The full list of commands accepted by this bot can be found here.

The pull request process is described here

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

@k8s-ci-robot k8s-ci-robot merged commit 7dd73c1 into kubernetes:release-1.14 Aug 11, 2019

5 of 7 checks passed

tide Not mergeable. Needs approved, lgtm labels.
Details
Pages changed 2427 new files uploaded
Details
Header rules 1 header rule processed
Details
Mixed content No mixed content detected
Details
Redirect rules 1 redirect rule processed
Details
cla/linuxfoundation xichengliudui authorized
Details
deploy/netlify Deploy preview ready!
Details
Alpha Disclaimer: this command is currently alpha.
-->
为前端代理生成客户端,并将其保存到 front-proxy-client.cert 和 front-proxy-client.key 中。

This comment has been minimized.

Copy link
@chenrui333

chenrui333 Aug 11, 2019

Member

声明是文件比较合适。

This comment has been minimized.

Copy link
@chenrui333

chenrui333 Aug 11, 2019

Member

并将其保存到 front-proxy-client.cert 文件和 front-proxy-client.key 文件中。


如果两个文件都已存在,kubeadm 将跳过生成步骤并将使用现有文件。

Alpha 免责声明:此命令目前是 alpha 阶段

This comment has been minimized.

Copy link
@chenrui333

chenrui333 Aug 11, 2019

Member

结尾要有句号。

@chenrui333

This comment has been minimized.

Copy link
Member

commented Aug 11, 2019

@markthink please give us some time to review the doc.

<td colspan="2">--cert-dir string&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;默认:"/etc/kubernetes/pki"</td>
</tr>
<tr>
<td></td><td style="line-height: 130%; word-wrap: break-word;">保存和存储证书的路径。</td>

This comment has been minimized.

Copy link
@chenrui333

chenrui333 Aug 11, 2019

Member

保存已经包含了存储的意思,其实。

@chenrui333

This comment has been minimized.

Copy link
Member

commented Aug 11, 2019

made couple of suggestions even though the PR has been merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.