-
Notifications
You must be signed in to change notification settings - Fork 146
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
Updated bundle parameter to include default array #687
Conversation
Putting this in draft; although it is shown to work in testing there was mention by @brooke-hamilton that we need to include guidance in our documentation for deployment instructions in other clouds to manually enable Pricings for Defender. |
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.
Thanks for this PR. ✨ The empty array looks good. Did you discover that it's not possible to retrieve the list of bundles during the deployment?
I'm running a test deployment but here are my initial comments.
…nto issue-649-temp
…nto issue-649-temp
e3791a3 addressed requested changes
Description
added in a default empty array value that will not issue a failure if the Microsoft.Security/pricings option is not available in region
Issue reference
The issue this PR will close: #649
Checklist
Please make sure you've completed the relevant tasks for this PR out of the following list: