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

Add new DNS API version #4583

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

alexwilcox9
Copy link

Adds support for DS and TLSA records as well as DNSSEC support

Used in hashicorp/terraform-provider-azurerm#28213

Copy link

hashicorp-cla-app bot commented Dec 9, 2024

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@stephybun stephybun left a comment

Choose a reason for hiding this comment

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

Thanks for this PR @alexwilcox9. Preview versions have a tendency to contain unexpected breaking changes, whether we allow preview versions to be used or not depends on several factors e.g. how popular/heavily used is the service, how pivotal is it in a customer's environment/infrastructure.

dns is a service we would want to be really careful about not introducing unforseen breaking changes so I've asked our counterparts at Microsoft to reach out to the service team for information on whether there are plans for this to go into a stable version (in which case we would prefer to wait for that to happen instead of introducing this) and whether they can provide some assurances that this won't contain any breaking changes going forwards.

I'll let you know once we have some feedback and whether we can proceed with this.

@stephybun stephybun self-assigned this Dec 11, 2024
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