Skip to content

Add something about Tencent Cloud#15298

Merged
k8s-ci-robot merged 11 commits intokubernetes:masterfrom
yunxji:master
Jul 26, 2019
Merged

Add something about Tencent Cloud#15298
k8s-ci-robot merged 11 commits intokubernetes:masterfrom
yunxji:master

Conversation

@yunxji
Copy link
Copy Markdown
Contributor

@yunxji yunxji commented Jul 5, 2019

Added Tencent Cloud Production Environment Solution

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jul 5, 2019
@k8s-ci-robot k8s-ci-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. language/en Issues or PRs related to English language sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels Jul 5, 2019
@netlify
Copy link
Copy Markdown

netlify bot commented Jul 5, 2019

Deploy preview for kubernetes-io-master-staging ready!

Built with commit 87649d2

https://deploy-preview-15298--kubernetes-io-master-staging.netlify.com

@yunxji
Copy link
Copy Markdown
Contributor Author

yunxji commented Jul 5, 2019

/assign @jaredbhatti

@yunxji yunxji changed the title Added Tencent Cloud Production Environment Solution Add something about Tengxun Cloud Jul 5, 2019
…: content/en/docs/setup/production-environment/turnkey/tke.md
@k8s-ci-robot k8s-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jul 5, 2019
@yunxji yunxji changed the title Add something about Tengxun Cloud Add something about Tencent Cloud Jul 5, 2019
Copy link
Copy Markdown
Contributor

@sftim sftim left a comment

Choose a reason for hiding this comment

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

Informal feedback

@yunxji
Copy link
Copy Markdown
Contributor Author

yunxji commented Jul 12, 2019

@jaredbhatti @mikedanese @brendanburns
May I know when the PR can be reviewed?

@yunxji
Copy link
Copy Markdown
Contributor Author

yunxji commented Jul 12, 2019

@sftim Thank you very much for your suggestions on the document. I have already revised it according to your suggestions. Please help advance this PR review.

@tengqm
Copy link
Copy Markdown
Contributor

tengqm commented Jul 15, 2019

I believe the text needs some improvement from native speakers. For example, the following sentence reads pretty strange to me:

For learn more about TKE in its official TKE documentation.


[Tencent Cloud Tencent Kubernetes Engine (TKE)](https://intl.cloud.tencent.com/product/tke) provides native Kubernetes container management services. You can deploy and manage a Kubernetes cluster with TKE in just a few steps. For detailed directions, see [Deploy Tencent Kubernetes Engine](https://intl.cloud.tencent.com/document/product/457/11741).

In addition to full compatibility with the native Kubernetes API,TKE features Kubernetes plugins of Tencent Cloud such as [Cloud Block Storage](https://intl.cloud.tencent.com/product/cbs) and [Cloud Load Balancer](https://intl.cloud.tencent.com/product/clb), helping you build a complete system of container services and functions. For learn more about TKE in its official [TKE documentation](https://intl.cloud.tencent.com/document/product/457).
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.

Suggested change
In addition to full compatibility with the native Kubernetes API,TKE features Kubernetes plugins of Tencent Cloud such as [Cloud Block Storage](https://intl.cloud.tencent.com/product/cbs) and [Cloud Load Balancer](https://intl.cloud.tencent.com/product/clb), helping you build a complete system of container services and functions. For learn more about TKE in its official [TKE documentation](https://intl.cloud.tencent.com/document/product/457).
In addition to full compatibility with the native Kubernetes API,TKE features Kubernetes plugins of Tencent Cloud such as [Cloud Block Storage](https://intl.cloud.tencent.com/product/cbs) and [Cloud Load Balancer](https://intl.cloud.tencent.com/product/clb), helping you build a complete system of container services and functions. To learn more, see the official [TKE documentation](https://intl.cloud.tencent.com/document/product/457).

@yunxji: What do you think about moving “To learn more…” into a What's Next? section.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

image
@sftim Excuse me, is this what you mean?

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.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@sftim Sorry, I have some misunderstandings about your suggestion before. Now I have made the following changes based on your suggestions. Please review
image

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@sftim

A What's Next section is like this example:
https://github.com/kubernetes/website/blame/abcee2dccd17c94376595b34cc05e56345a7f06c/content/zh/docs/concepts/workloads/pods/podpreset.md#L66

Is this the case? If this is possible, please help merge this PR.

image


[Tencent Cloud Tencent Kubernetes Engine (TKE)](https://intl.cloud.tencent.com/product/tke) provides native Kubernetes container management services. You can deploy and manage a Kubernetes cluster with TKE in just a few steps. For detailed directions, see [Deploy Tencent Kubernetes Engine](https://intl.cloud.tencent.com/document/product/457/11741).

In addition to full compatibility with the native Kubernetes API,TKE features Kubernetes plugins of Tencent Cloud such as [Cloud Block Storage](https://intl.cloud.tencent.com/product/cbs) and [Cloud Load Balancer](https://intl.cloud.tencent.com/product/clb), helping you build a complete system of container services and functions.
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.

nit:

Suggested change
In addition to full compatibility with the native Kubernetes API,TKE features Kubernetes plugins of Tencent Cloud such as [Cloud Block Storage](https://intl.cloud.tencent.com/product/cbs) and [Cloud Load Balancer](https://intl.cloud.tencent.com/product/clb), helping you build a complete system of container services and functions.
In addition to full compatibility with the native Kubernetes API, TKE features Kubernetes plugins for Tencent Cloud such as [Cloud Block Storage](https://intl.cloud.tencent.com/product/cbs) and [Cloud Load Balancer](https://intl.cloud.tencent.com/product/clb), helping you build a complete system of container services and functions.

If TKE is a certified solution, mention that and simplify the text around API compatibility. At the moment this reads too much like marketing text.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@sftim Modified according to your suggestions. Please review.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@sftim What is your opinion on these changes? Is it suitable?
image


The core of Tencent Kubernetes Engine is open source and available [on GitHub](https://github.com/TencentCloud/tencentcloud-cloud-controller-manager/).

When using TKE to create a Kubernetes cluster, you can choose managed mode or independent deployment mode. In addition, you can customize the deployment as needed; for example, you can choose an existing CVM instance for cluster creation or enable Kube-proxy IPVS.
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.

Suggested change
When using TKE to create a Kubernetes cluster, you can choose managed mode or independent deployment mode. In addition, you can customize the deployment as needed; for example, you can choose an existing CVM instance for cluster creation or enable Kube-proxy IPVS.
When using TKE to create a Kubernetes cluster, you can choose managed mode or independent deployment mode. In addition, you can customize the deployment as needed; for example, you can choose an existing CVM instance for cluster creation or enable Kube-proxy in IPVS mode.

Copy link
Copy Markdown
Contributor

@sftim sftim left a comment

Choose a reason for hiding this comment

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

@yunxji
Copy link
Copy Markdown
Contributor Author

yunxji commented Jul 18, 2019

This page should be Hugo markdown, with front matter similar to https://github.com/kubernetes/website/blame/5f27e08983d12a183ea99b599b9b09eeaa341bb3/content/en/docs/setup/production-environment/turnkey/aws.md#L1-L7

Does that make sense?

@sftim Thank you for reminding me. I can't say what the meaning is, but I think it's necessary. Just a little uncertain is that I don’t know if the reviewers added the right ones. What do you think?
image

@yunxji
Copy link
Copy Markdown
Contributor Author

yunxji commented Jul 18, 2019

@sftim Can you help merge this PR?

@sftim
Copy link
Copy Markdown
Contributor

sftim commented Jul 18, 2019

I'm also not sure about what to put for the reviewers on this page I'm afraid.
Maybe ask for advice via SIG Docs' Slack channel?

Apart from the detail about reviewers, the PR looks good to me.

@yunxji
Copy link
Copy Markdown
Contributor Author

yunxji commented Jul 19, 2019

The answer I got for this question is:

This page should be Hugo markdown, with front matter similar to https://github.com/kubernetes/website/blame/5f27e08983d12a183ea99b599b9b09eeaa341bb3/content/en/docs/setup/production-environment/turnkey/aws.md#L1-L7

Does that make sense?

@sftim
The answer I got for this question is:The tags are for Hugo to organize contents and for related Bots to trigger notifications etc.

Can I keep this content and request a merge?

Copy link
Copy Markdown
Contributor Author

@yunxji yunxji left a comment

Choose a reason for hiding this comment

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

@sftim Maybe this is more appropriate. Please help merge it.
image

@sftim
Copy link
Copy Markdown
Contributor

sftim commented Jul 22, 2019

/lgtm

At this point @yunxji you're waiting on approval from this PR's assignee, who I think is @jaredbhatti

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jul 22, 2019
@yunxji
Copy link
Copy Markdown
Contributor Author

yunxji commented Jul 23, 2019

Request the /approve tag. @jaredbhatti

@yunxji
Copy link
Copy Markdown
Contributor Author

yunxji commented Jul 25, 2019

/assign @tengqm

@yunxji
Copy link
Copy Markdown
Contributor Author

yunxji commented Jul 25, 2019

pinging @tengqm for approval

@yunxji
Copy link
Copy Markdown
Contributor Author

yunxji commented Jul 26, 2019

pinging @jaredbhatti for approval

@tengqm
Copy link
Copy Markdown
Contributor

tengqm commented Jul 26, 2019

/approve

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: tengqm

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

The pull request process is described 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

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 26, 2019
@k8s-ci-robot k8s-ci-robot merged commit c866b20 into kubernetes:master Jul 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/en Issues or PRs related to English language lgtm "Looks good to me", indicates that a PR is ready to be merged. sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants