-
Notifications
You must be signed in to change notification settings - Fork 87
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
improve (dns): support domain group manage and modify ali_dns_domain #202
Conversation
0e17e53
to
b32303b
Compare
""" | ||
|
||
RETURN = ''' | ||
dns: |
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.
返回的不应该是dns group么?
- Create, Delete Alicloud cloud DNS group(DNS group). | ||
It supports updating group name and domain group. | ||
options: | ||
domain_name: |
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.
这个参数是干啥的
- Create, Delete Alicloud cloud DNS group(DNS group). | ||
It supports updating group name. | ||
options: | ||
group_id: |
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.
group id 用来干啥
- This module fetches data from the Open API in Alicloud. | ||
The module must be called from within the group itself. | ||
options: | ||
domain_count: |
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.
domain count 用来干啥
b764738
to
809dbbb
Compare
- Whether or not to create, delete DNS group. | ||
choices: ['present', 'absent'] | ||
default: 'present' | ||
new_group_name: |
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.
new group name 去掉吧,这个name就不支持修改了
try: | ||
for v in dns_conn.describe_domain_groups(): | ||
if v.name == group_name: | ||
matching_dns.append(v) |
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.
我理解不用matching_dns了,找到了直接返回就行
- This module fetches data from the Open API in Alicloud. | ||
The module must be called from within the group itself. | ||
options: | ||
domain_count: |
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.
这个domain count直接去掉就行了
No description provided.