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 Azure Firewall custom DNS servers as parameter and resource setting #626

Closed
2 tasks done
amp-pds opened this issue Sep 5, 2023 · 1 comment · Fixed by #615
Closed
2 tasks done

Add Azure Firewall custom DNS servers as parameter and resource setting #626

amp-pds opened this issue Sep 5, 2023 · 1 comment · Fixed by #615
Labels
Awaiting Release To Be Cut ✂️ This is fixed in the main branch but not in the latest release, will be fixed with next release cut

Comments

@amp-pds
Copy link

amp-pds commented Sep 5, 2023

Describe the feature end to end, including deployment scenario details under which the feature would occur.

Request to include an additional parameter in the vwanConnectivity.bicep module to update the Azure Firewall with custom DNS server IP(s) and the parameter/setting to be included in the resource resFirewallPolicies deployment.

Why is this feature important. Describe why this would be important for your organization and others. Would this impact similar orgs in the same way?

If using a Private DNS Resolver service or own DNS virtual machines with Azure Firewall DNS proxy enabled, then need a way to pass the IP address(es) to be added as custom DNS servers on the Azure Firewall.

Please provide the correlation id associated with your error or bug.

xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx

Can you describe any alternatives that you have taken since this feature does not exist?

No response

Feature Implementation

No response

Check previous GitHub issues

  • I have searched the issues for this item and found no duplicate

Code of Conduct

  • I agree to follow this project's Code of Conduct
@jtracey93
Copy link
Collaborator

Hey @amp-pds,

This is actually already in a PR here #615 that @juang903 is working on and its nearly ready to be merged, just some linting to sort 👍

Stay tuned

@jtracey93 jtracey93 linked a pull request Sep 5, 2023 that will close this issue
@jtracey93 jtracey93 added the Awaiting Release To Be Cut ✂️ This is fixed in the main branch but not in the latest release, will be fixed with next release cut label Sep 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Release To Be Cut ✂️ This is fixed in the main branch but not in the latest release, will be fixed with next release cut
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants