-
Notifications
You must be signed in to change notification settings - Fork 53
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
docs(cce): fix docs issues. #994
docs(cce): fix docs issues. #994
Conversation
30b6637
to
c1507b3
Compare
docs/data-sources/cce_clusters.md
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clusters.masters.availability_zone
is missing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Referencing Huawei Cloud, not processed temporarily
docs/resources/cce_node_pool_v3.md
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
scall_enable
and extend_param
are missing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
scall_enable
is deprecated
extend_param
declared in schema only, but it is not used
docs/data-sources/cce_node_v3.md
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The validity of the arguments needs to be tested.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unit test has but docs miss the field
docs/resources/cce_addon_v3.md
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this docs have no field which is new added
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unit test has but docs miss the field
edc4287
to
b1aa225
Compare
docs/data-sources/cce_clusters.md
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clusters.highway_subnet_id
does not exist in schema
384b35b
to
f1a55ee
Compare
/lgtm |
docs/data-sources/cce_cluster_v3.md
Outdated
|
||
* `id` - (Optional) The ID of container cluster. | ||
* `name` - (Optional, String)The Name of the cluster resource. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* `name` - (Optional, String)The Name of the cluster resource. | |
* `name` - (Optional, String)The Name of the cluster resource. |
f1a55ee
to
7d6f5bb
Compare
fix docs issues.
data_source_flexibleengine_cce_clusters
1、add attribute field