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

Bastion NSG Conditional Deployment #606

Merged
merged 2 commits into from
Aug 10, 2023
Merged

Bastion NSG Conditional Deployment #606

merged 2 commits into from
Aug 10, 2023

Conversation

4pplied
Copy link
Contributor

@4pplied 4pplied commented Aug 10, 2023

Overview/Summary

Adding a if condition for the Bastion NSG to be deployed if the parAzBastionEnabled is set to true.

This PR fixes/adds/changes/removes

  1. if condition for Bastion NSG to be deployed if Bastion Enablement is set to true.

Breaking Changes

  1. N/A

Testing Evidence

image

As part of this Pull Request I have

@4pplied
Copy link
Contributor Author

4pplied commented Aug 10, 2023

One to note is that the Hub Virtual Network has a dependsOn for the NSG. I tested the if condition for parAzBastionEnabled to be false and all deployed still fine as screenshots above.

@jtracey93 jtracey93 merged commit 800ae63 into Azure:main Aug 10, 2023
7 checks passed
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