-
Notifications
You must be signed in to change notification settings - Fork 326
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
adding logic to check if Key Vault or SQL server exists. #579
adding logic to check if Key Vault or SQL server exists. #579
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has some of your environment and not the params.
deployment/Deploy.ps1
Outdated
Write-Host "To Purge KeyVault please use the following doc:" | ||
Write-Host "https://learn.microsoft.com/en-us/cli/azure/keyvault?view=azure-cli-latest#az-keyvault-purge." | ||
Write-Host "You could use new KeyVault name by using parameter -KeyVault" | ||
Throw "🛑 KeyVault name already exists." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do not throw the error, instead provide the message as is and exit with 1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@msalemcode Its all good... just add an extra space on the emoji and merge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works fine with new install and new installs with pre-existing artifacts.
@msalemcode - Code approved, pleases merge when ready. |
Adding logic to deployment script to exit if Key Vault exists and deleted and give information how to proceed.