-
Notifications
You must be signed in to change notification settings - Fork 460
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
Update Azure DevTest Lab API to "2018-10-15-preview" #2523
Conversation
…ure/ResourceModules into users/ahmad/2518_dtlApiUpdate
Test Results 1 files ± 0 1 suites ±0 1m 44s ⏱️ + 1m 23s Results for commit f9cbf69. ± Comparison against base commit 254a756. This pull request removes 47 and adds 274 tests. Note that renamed tests count towards both.
This pull request removes 1 skipped test and adds 7 skipped tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
modules/Microsoft.DevTestLab/labs/notificationChannels/deploy.bicep
Outdated
Show resolved
Hide resolved
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.
The API Specs file was updated 3 days ago. If the latest logic works - it should be safe to undo the changes now ad they 'should' not be applied anymore
Co-authored-by: Alexander Sehr <ASehr@hotmail.de>
Undo complete :) . |
Description
On the back of #2518
See documentation for more details
Added a custom logic in the removal script dedicated for DevTest Lab use case:
Added cost child module.
Updated notification channel child resource - name to have allowed names: autoshutdown, costThreshold, as per the API documentation examples:
Pipeline references
Type of Change
Please delete options that are not relevant.
Checklist