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

improve (dns): support domain group manage and modify ali_dns_domain #202

Merged
merged 1 commit into from
Dec 3, 2019

Conversation

lixue323
Copy link
Contributor

No description provided.

@lixue323 lixue323 changed the title improve (dns): support domain group manage improve (dns): support domain group manage and modify ali_dns_domain Nov 21, 2019
@lixue323 lixue323 force-pushed the dns branch 4 times, most recently from 0e17e53 to b32303b Compare November 22, 2019 08:44
"""

RETURN = '''
dns:
Copy link
Collaborator

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:
Copy link
Collaborator

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:
Copy link
Collaborator

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:
Copy link
Collaborator

Choose a reason for hiding this comment

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

domain count 用来干啥

@lixue323 lixue323 force-pushed the dns branch 4 times, most recently from b764738 to 809dbbb Compare December 2, 2019 10:37
- Whether or not to create, delete DNS group.
choices: ['present', 'absent']
default: 'present'
new_group_name:
Copy link
Collaborator

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)
Copy link
Collaborator

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:
Copy link
Collaborator

Choose a reason for hiding this comment

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

这个domain count直接去掉就行了

@xiaozhu36 xiaozhu36 merged commit b7977bf into alibaba:master Dec 3, 2019
@lixue323 lixue323 deleted the dns branch December 3, 2019 07:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants