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

azurerm_dns_zone & azurerm_private_dns_zone - support for the soa_record block #9319

Merged
merged 12 commits into from Nov 19, 2020

Conversation

neil-yechenwei
Copy link
Contributor

Copy link
Collaborator

@katbyte katbyte left a comment

Choose a reason for hiding this comment

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

Thanks @neil-yechenwei - given that you cannot update or delete SOA records can we mark it and its properties as force new?

@neil-yechenwei
Copy link
Contributor Author

@katbyte , thanks for your comments. I've updated code. Please have a look. Thanks.

Copy link
Collaborator

@katbyte katbyte left a comment

Choose a reason for hiding this comment

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

Thanks @neil-yechenwei - left a couple more comments inline

@neil-yechenwei
Copy link
Contributor Author

@katbyte , thanks for your comments. I've updated code. Please have a look. Thanks.

@katbyte katbyte added this to the v2.37.0 milestone Nov 19, 2020
Copy link
Collaborator

@katbyte katbyte left a comment

Choose a reason for hiding this comment

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

Thanks @neil-yechenwei - this LGTM now 👍

@katbyte katbyte changed the title Add support for DNS SOA Record and Private DNS SOA Record azurerm_dns_zone & azurerm_private_dns_zone - support for the soa_record block Nov 19, 2020
@katbyte katbyte merged commit a0155e2 into hashicorp:master Nov 19, 2020
katbyte added a commit that referenced this pull request Nov 19, 2020
@ghost
Copy link

ghost commented Nov 20, 2020

This has been released in version 2.37.0 of the provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. As an example:

provider "azurerm" {
    version = "~> 2.37.0"
}
# ... other configuration ...

@delampri
Copy link

I see no documentation for this feature on provider 2.37 https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs

@tombuildsstuff
Copy link
Member

@neil-yechenwei can you send a PR to add this?

@neil-yechenwei
Copy link
Contributor Author

neil-yechenwei commented Nov 29, 2020

@tombuildsstuff, seems I‘ve documented soa_record in azurerm_private_dns_zone and azurerm_dns_zone, right? If it's not expected, please correct me. Thanks.

@ghost
Copy link

ghost commented Dec 19, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks!

@hashicorp hashicorp locked as resolved and limited conversation to collaborators Dec 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants