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.

Sign up for GitHub

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

Jump to bottom

No support for DNS alias record sets #467

Open
tlourey opened this issue Mar 24, 2021 · 4 comments · May be fixed by #1767
Open

No support for DNS alias record sets #467

tlourey opened this issue Mar 24, 2021 · 4 comments · May be fixed by #1767
Labels
has_pr PR fixes have been made medium_priority Medium priority new_feature New feature requirments

Comments

@tlourey
Copy link

tlourey commented Mar 24, 2021

SUMMARY

No support for DNS alias record sets aka target resource.

ISSUE TYPE
  • Bug Report
COMPONENT NAME

azure_rm_dnsrecordset
azure_rm_publicipaddress

ANSIBLE VERSION
ansible 2.10.2
  config file = None
  configured module search path = ['/home/tim/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /opt/ansible/lib/python3.7/site-packages/ansible
  executable location = /opt/ansible/bin/ansible
  python version = 3.7.3 (default, Jul 25 2020, 13:03:44) [GCC 8.3.0]

ansible 2.10.7
  config file = /home/azureuser/.ansible.cfg
  configured module search path = ['/home/azureuser/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/lib/python3.6/site-packages/ansible
  executable location = /usr/local/bin/ansible
  python version = 3.6.8 (default, Nov 16 2020, 16:55:22) [GCC 4.8.5 20150623 (Red Hat 4.8.5-44)]
CONFIGURATION
OS / ENVIRONMENT

Azure Cloud Shell
CentOS Linux release 7.9.2009 (Core)

STEPS TO REPRODUCE

None - option not documented on : https://docs.ansible.com/ansible/latest/collections/azure/azcollection/azure_rm_dnsrecordset_module.html#parameters

On https://docs.ansible.com/ansible/latest/collections/azure/azcollection/azure_rm_publicipaddress_module.html#parameters there is an option for domain_name but this only covers 'The customizable portion of the FQDN assigned to public IP address'

- name: ensure an "A" record set with multiple records
  azure_rm_dnsrecordset:
    resource_group: myResourceGroup
    relative_name: www
    zone_name: testing.com
    record_type: A
    targetResouce: "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network//providers/Microsoft.Network/dnsZones/zonename/A/recordname"
- name: Create a public ip address
  azure_rm_publicipaddress:
    resource_group: myResourceGroup
    name: my_public_ip
    allocation_method: static
    targetzone: "subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Network/dnsZones/{zoneName}"
EXPECTED RESULTS

DNS Record created and linked to Alias object

ACTUAL RESULTS

Doesn't work

References:
https://docs.microsoft.com/en-us/rest/api/dns/recordsets/createorupdate#create-a-recordset-with-alias-target-resource
https://docs.microsoft.com/en-us/rest/api/dns/recordsets/createorupdate#subresource
https://docs.microsoft.com/en-us/rest/api/virtualnetwork/publicipaddresses/createorupdate#subresource
@Fred-sun
Copy link
Collaborator

@tlourey Thank you for reporting the problem. We'll look into it as soon as possible. Thank you very much!

@Fred-sun Fred-sun added the medium_priority Medium priority label Mar 25, 2021
@Fred-sun Fred-sun added the work in In trying to solve, or in working with contributors label Jul 14, 2021
@trolleyboy
Copy link

Was just looking for this today. It would be great to see azure_rm_dnsrecordset updated with support for Azure DNS Alias records.

@Fred-sun
Copy link
Collaborator

@tlourey Sorry for not updating in time, we will deal with this problem as soon as possible. Thank you very much!

@Fred-sun Fred-sun linked a pull request Nov 22, 2024 that will close this issue
Open
@Fred-sun Fred-sun added has_pr PR fixes have been made new_feature New feature requirments and removed work in In trying to solve, or in working with contributors labels Nov 27, 2024
@Fred-sun
Copy link
Collaborator

@tlourey Added in #1767

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
has_pr PR fixes have been made medium_priority Medium priority new_feature New feature requirments
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add target_resource to azure_rm_dnsrecoredset.py Fred-sun/ansible_collections_azure
3 participants
@tlourey @Fred-sun @trolleyboy