DNS server Support for Bloxone Ansible v2 #49
Conversation
| elements: dict | ||
| suboptions: | ||
| algorithm: | ||
| description: "" |
There was a problem hiding this comment.
Please add description to all the algorithm fields.
| - address: "192.168.11.11" | ||
| fqdn: "example.com." | ||
| tags: | ||
| location: "my-location" |
There was a problem hiding this comment.
| location: "my-location" | |
| location: "site-1" |
| infoblox.bloxone.dns_server_info: | ||
| id: "{{ dns_server_id }}" | ||
|
|
||
| - name: Get DNS Server information by filters (e.g., name) |
There was a problem hiding this comment.
| - name: Get DNS Server information by filters (e.g., name) | |
| - name: Get DNS Server information by filters (e.g. name) |
| - dns_server_info.objects[0].id == dns_server.id | ||
| - dns_server_info.objects[0].add_edns_option_in_outgoing_query == true | ||
|
|
||
| - name: Create a DNS Server with auto_sort_views enabled |
There was a problem hiding this comment.
| - name: Create a DNS Server with auto_sort_views enabled | |
| - name: Create a DNS Server with Auto Sort Views enabled |
| - name: Create a DNS Server with Custom Root Nameservers | ||
| infoblox.bloxone.dns_server: | ||
| name: "{{ dns_server_name }}" | ||
| custom_root_ns: |
There was a problem hiding this comment.
Can you add multiple name servers here ?
| - dns_server_info is not failed | ||
| - dns_server_info.objects | length == 1 | ||
| - dns_server_info.objects[0].recursion_enabled == true | ||
|
|
There was a problem hiding this comment.
Missing UT for recursive_clients attribute.
| - dns_server_info.objects | length == 1 | ||
| - dns_server_info.objects[0].recursion_enabled == true | ||
|
|
||
| - name: Create a DNS server with resolver_query_timeout set to 15 |
There was a problem hiding this comment.
Please you modify all the task names to include attribute names in Capital case and not snake case and remove the value of the attribute .
| - dns_server_info.objects | length == 1 | ||
| - dns_server_info.objects[0].secondary_soa_query_limit == 2 | ||
|
|
||
| - name: Create a DNS server with sort_list configuration |
There was a problem hiding this comment.
Can you test any value for the element in sort list ?
| state: present | ||
| register: dns_server | ||
|
|
||
| - name: Get Information about the DNS Server |
There was a problem hiding this comment.
| - name: Get Information about the DNS Server | |
| - name: Get Information about the DNS Server by filter (name) |
| infoblox.bloxone.dns_server: | ||
| name: "{{ dns_server_name }}" | ||
| state: "absent" | ||
| ignore_errors: true |
There was a problem hiding this comment.
Please add a new line at the end of file
Sanity - passed
integration - passed
lint check - passed