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

[PrivateDns] Add resolutionPolicy property to vnet links #30115

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

Conversation

pranavpnair
Copy link

@pranavpnair pranavpnair commented Oct 17, 2024

Related command

network private-dns link vnet create
network private-dns link vnet delete
network private-dns link vnet list
network private-dns link vnet show
network private-dns link vnet update
network private-dns link vnet wait

Description

In this change, we add a new property to (private dns) vnet link resources: resolutionPolicy. This will be an optional parameter in CLI. It is an enum that consists of two possible values - "Default" or "NxDomainRedirect".
For example, to create a new vnet link with resolutionPolicy property set to "NxDomainRedirect", one can run the following command:
az network private-dns link vnet create -g myResourceGroup -n myLink -z privatelink.consoto.com -v {vnet-resourceId} -e True/False --resolution-policy NxDomainRedirect

This property has already been added to swagger and other language SDKs and publicly released.

Swagger change: Azure/azure-rest-api-specs#29470

New property can be found here

.NET sdk reference documentation can be found here

Pull request on Aaz repo: Azure/aaz#591

Testing Guide

Added new tests and generated yaml files.

History Notes


This checklist is used to make sure that common guidelines for a pull request are followed.

Copy link

Validation for Azure CLI Full Test Starting...

Thanks for your contribution!

Copy link

Validation for Breaking Change Starting...

Thanks for your contribution!

@yonzhan
Copy link
Collaborator

yonzhan commented Oct 17, 2024

Thank you for your contribution! We will review the pull request and get back to you soon.

@pranavpnair pranavpnair changed the title Add resolutionPolicy param to vnet links {Network} Add resolutionPolicy param to vnet links Oct 17, 2024
@pranavpnair pranavpnair changed the title {Network} Add resolutionPolicy param to vnet links [PrivateDns] Add resolutionPolicy param to vnet links Oct 17, 2024
@pranavpnair pranavpnair changed the title [PrivateDns] Add resolutionPolicy param to vnet links [PrivateDns] Add resolutionPolicy property to vnet links Oct 17, 2024
@pranavpnair pranavpnair marked this pull request as ready for review October 17, 2024 03:06
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.

3 participants