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

azure_rm_dnsrecordset: fix updating SOA record #1052

Merged
merged 4 commits into from
Jan 12, 2023

Conversation

akira6592
Copy link
Contributor

@akira6592 akira6592 commented Dec 21, 2022

SUMMARY

fix #1047

fix type convert error in updating SOA record.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME
  • azure_rm_dnsrecordset
ADDITIONAL INFORMATION

integration test works fine in my local as bellow.

TASK [azure_rm_dnsrecordset : Update SOA record] *******************************
changed: [testhost] => {"changed": true, "state": {"etag": "xxxxx", "fqdn": "480b92c2cae5ec3d25349.com.", "id": "/subscriptions/xxxxx/resourceGroups/myrg/providers/Microsoft.Network/dnszones/480b92c2cae5ec3d25349.com/SOA/@", "name": "@", "provisioning_state": "Succeeded", "soa_record": {"email": "azuredns-hostmaster99.example.com", "expire_time": 2419299, "host": "ns1-99.example.com.", "minimum_ttl": 399, "refresh_time": 3699, "retry_time": 399, "serial_number": 99}, "target_resource": {}, "ttl": 3600, "type": "SOA"}}

@Fred-sun Fred-sun added ready_for_review The PR has been modified and can be reviewed and merged medium_priority Medium priority labels Dec 22, 2022
@xuzhang3
Copy link
Collaborator

@akira6592 LGTM

@xuzhang3 xuzhang3 merged commit 7965032 into ansible-collections:dev Jan 12, 2023
@akira6592 akira6592 deleted the fix-dns branch January 12, 2023 12:36
@akira6592
Copy link
Contributor Author

Thank you for the merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
medium_priority Medium priority ready_for_review The PR has been modified and can be reviewed and merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

azure_rm_dnsrecordset can not update SOA record (unable to convert to long)
3 participants