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

tencentcloud_kubernetes_cluster add new cluster_os #200

Merged
merged 1 commit into from
Nov 19, 2019
Merged

tencentcloud_kubernetes_cluster add new cluster_os #200

merged 1 commit into from
Nov 19, 2019

Conversation

Himer
Copy link
Contributor

@Himer Himer commented Nov 19, 2019

Resource: tencentcloud_kubernetes_cluster cluster_os add new support: centos7.6x86_64 and ubuntu18.04.1 LTSx86_64
#199

…t: `centos7.6x86_64` and `ubuntu18.04.1 LTSx86_64`
@@ -132,7 +132,7 @@ The following arguments are supported:
* `cluster_max_pod_num` - (Optional, ForceNew) The maximum number of Pods per node in the cluster. Default is 256. Must be a multiple of 16 and large than 32.
* `cluster_max_service_num` - (Optional, ForceNew) The maximum number of services in the cluster. Default is 256. Must be a multiple of 16.
* `cluster_name` - (Optional, ForceNew) Name of the cluster.
* `cluster_os` - (Optional, ForceNew) Operating system of the cluster, the available values include: 'centos7.2x86_64' and 'ubuntu16.04.1 LTSx86_64'. Default is 'ubuntu16.04.1 LTSx86_64'.
* `cluster_os` - (Optional, ForceNew) Operating system of the cluster, the available values include: 'centos7.2x86_64','centos7.6x86_64','ubuntu16.04.1 LTSx86_64','ubuntu18.04.1 LTSx86_64'. Default is 'ubuntu16.04.1 LTSx86_64'.
Copy link
Contributor

Choose a reason for hiding this comment

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

逗号后面没空格,没有 and

Copy link
Contributor Author

Choose a reason for hiding this comment

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

这又不是mardown语法,没那么严格

@Himer Himer merged commit 16c7856 into tencentcloudstack:master Nov 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants